Please upgrade here. These earlier versions are no longer being updated and have security issues.
HackerOne users: Testing against this community violates our program's Terms of Service and will result in your bounty being denied.

Disabling Conversations application doesn't remove Inbox

edited April 2011 in Vanilla 2.0 - 2.8
This may be a bug.

I can manually remove it from the template, but since I'm using the tag {inbox} I would have expected the tag to be smart about not generating anything if the application is disabled. Because application is disabled and the link exists it results in a 404 error.

Comments

  • lucluc ✭✭
    edited April 2011
    The tag should be inbox_link though.

    But indeed, it doesn't seem to test anything:
    function smarty_function_inbox_link($Params, &$Smarty) {
    $Wrap = GetValue('wrap', $Params, 'li');
    return Gdn_Theme::Link('inbox',
    GetValue('text', $Params, T('Inbox')),
    GetValue('format', $Params, Wrap('%text', $Wrap)));
    }
    The link seems to be added by the application code, so I guess it's safe to remove it from any theme.
  • hello,
    I am having the same trouble with the version of Vanilla. Can you tell me where the code is to delete so the Inbox will magically disappear?

    Thank you.
Sign In or Register to comment.