Advise Guest to login to see Attachments

[aw06]

OK, that didn't work lol

[Matthew K.]

No, it should be && $context['user']['is_logged']. If you look at the code inside the if statement, it's to SHOW attachments...and you only want them to show if there ARE attachments, and it's a member. Then, we add an elseif statement below that shows them a message if there ARE attachments, but they're a guest.

It shows one message despite the count of attachments. It shouldn't have any conflicts. It's fairly straight forward.

It makes two modifications to Display.template.php.

The 1st:

Code (Find) Select Expand

if (!empty($message['attachment']))

Code (Replace) Select Expand

if (!empty($message['attachment']) && $context['user']['is_logged'])

Should'nt that be
if (!empty($message['attachment']) && $context['user']['is_guest'])

?

[Matthew K.]

What about it didn't work?

OK, that didn't work lol

[aw06]

Not seeing the message ... attachments just disappear

[Matthew K.]

What edits specifically did you complete?

[aw06]

What edits specifically did you complete?

Those listed ... did them exactly.

It shows one message despite the count of attachments. It shouldn't have any conflicts. It's fairly straight forward.

It makes two modifications to Display.template.php.

The 1st:

Code (Find) Select Expand

if (!empty($message['attachment']))

Code (Replace) Select Expand

if (!empty($message['attachment']) && $context['user']['is_logged'])
The 2nd:

Code (Find) Select Expand

echo '
</td>
</tr><tr>
<td valign="bottom" class="smalltext" id="modified_', $message['id'], '">';

Code (Add Before) Select Expand

// Labradoodle-360; Hide Attachments from Guests
elseif ($context['user']['is_guest'] && !empty($message['attachment']))
echo '<hr width="100%" size="1" class="hrcolor" />
<div style="overflow: auto; width: 100%;">
<strong>', $txt['lab360_attachments_avail'], '</strong>
</div>';
// End Labradoodle-360; Hide Attachments from Guests
Then it makes one modification to Modifications.english.php

Code (Add Before End) Select Expand

$txt['lab360_attachments_avail'] = 'Attachment(s) Available - <a href="'. $scripturl. '?action=login" target="_self">Login</a> or <a href="'. $scripturl. '?action=register" target="_self">Register</a>';

Donations are appreciated as always

You can also take this mod over and submit it to the SMF Mod Site if you would like, as I have no intentions of submitting it.

Best Regards,
Labradoodle-360

[Matthew K.]

Try to figure out if the conditions are being met or not...you can simply use die(); after the if statement, and after the elseif, and see which one is returned.

Furthermore, are there any errors in the error log?

[aw06]

Try to figure out if the conditions are being met or not...you can simply use die(); after the if statement, and after the elseif, and see which one is returned.

Furthermore, are there any errors in the error log?

Negative .. no errors

[Matthew K.]

Try to figure out if the conditions are being met or not...you can simply use die(); after the if statement, and after the elseif, and see which one is returned.

[aw06]

as in ..

if (!empty($message['attachment']) && $context['user']['is_logged']) die();

and

elseif ($context['user']['is_guest'] && !empty($message['attachment'])) die();

??

[Matthew K.]

die(); would be on it's own line, but yeah. In addition, you'll probably want to put brackets around the if, and elseif statements.

Example

Code Select Expand

if (this)
{
// Do this
}
elseif (this)
{
// Do this
}

[aw06]

die(); would be on it's own line, but yeah. In addition, you'll probably want to put brackets around the if, and elseif statements.

Example

Code Select Expand

if (this)
{
// Do this
}
elseif (this)
{
// Do this
}

OK kool, thing is i am not a ninja coder like you guys  so i really am not 100% certain how the do the coding

[Matthew K.]

It's not a problem

[ziskar364]

I know I am digging in graves but is there any similar mod for SMF 2.0.17?

I have recently started allowing specific sections of the forum to be publicly visible but their attachments are hidden, so I would like to show a message to guests that will advise registering in order to view attachments and gain full access to the forum topics.

I would really appreciate any help!