Not signed in (Sign In)

Categories

Vanilla 1.1.5 is a product of Lussumo. More Information: Documentation, Community Support.

Help keep Vanilla free:
Welcome Guest!
Want to take part in these discussions? If you have an account, sign in now.
If you don't have an account, apply for one now.

Vanilla: Hiding the Edit button on comments

Bottom of Page

1 to 15 of 15

  1.  
  1.  # 1
    I'm new to all this Vanilla goodness, but I want to use it in my classroom. Please be very elementary in your answers so I know exactly how to implement it.

    How do I go about hiding the "edit" button from members (but not administrators)? Or is there a way to see all the different editions of a comment as an administrator? Because I'm working with middle school students, I have to worry about all the different ways they could use this great tool for cheating. They aren't able to delete their comments, so that's good, but I don't know how to stop them from posting the answers to an assignment, then later going back and changing the post to something harmless...

    As long as I have a way to see what all the editions of their post said, or disable the edit button...
    •  
      CommentAuthorWallPhone
    • CommentTimeSep 10th 2006
     # 2
    • CommentAuthorithcy
    • CommentTimeSep 10th 2006
     # 3
    that's a lot of backslashes
    • CommentAuthorithcy
    • CommentTimeSep 10th 2006
     # 4
    hey wallphone, i just had a look at your source.
    if ( $Comment->Context->Session->UserID < 1 && !$Comment->Context->Session->User->Permission('PERMISSION_EDIT_COMMENTS') )
    will this really work? it seems to be saying "if your userID is 0 (i.e, you're the admin), AND you don't have permission to edit all comments, then remove the edit links"
    shouldn't it be if ( $Comment->Context->Session->UserID > 0 || !$Comment->Context->Session->User->Permission('PERMISSION_EDIT_COMMENTS') )
    instead?
    or am i reading it wrong?
    • CommentAuthorithcy
    • CommentTimeSep 10th 2006
     # 5
    also:
    as wallphone notes in his addon, this can be bypassed rather easily. i don't think this functionality is fully possible right now without a theme/addon combination, and possibly even some changes to the core.
    •  
      CommentAuthorWallPhone
    • CommentTimeSep 10th 2006
     # 6
    Oops--errors fixed. One side effect is that the 'edited' flag also changes to 'ed'.

    I got this pulled from code of another extension I am wrking on currently. Will flesh this one out more when I am done with the others, possibly removing the 'hackable' ability. This will still at least give the functionality needed.
    • CommentAuthorithcy
    • CommentTimeSep 10th 2006 edited
     # 7
    One side effect is that the 'edited' flag also changes to 'ed'.

    well, hopefully no threads will be 'edited' anyway :)
  8.  # 8
    @ithcy: A UserID of 0 indicates that the user is not logged in.
    • CommentAuthorArturo G.
    • CommentTimeSep 11th 2006
     # 9
    @Wallphone, @ithcy: Is it very easy to break bypass attempts. I did it in the DiceRoller extension.

    This is a commented excerpt of the code I used:

    if (in_array($Context->SelfUrl, array("post.php"))) {
    // DELEGATION FUNCTION TO VALIDATE EDIT
    function DiscussionForm_DiceRoller_EditValidation( &$DiscussionForm ) {
    $pba = $DiscussionForm->PostBackAction;

    // PREVENT ACCESS TO THE EDIT FORM AND SAVE OPERATIONS FOR COMMENTS
    // WHICH MEET A CERTAIN CONDITION
    if ( $DiscussionForm->Comment->CommentID != 0
    && ($pba=='' || $pba=='SaveDiscussion' || $pba=='SaveComment') ) {
    // CHECK THE CONDITION
    if ( !$DiscussionForm->Context->Session->User->Permission('PERMISSION_EDIT_DISCUSSIONS') ) {
    // BREAK THE EDIT ATTEMPT
    $DiscussionForm->Context->WarningCollector->Add(
    $DiscussionForm->Context->GetDefinition("NoEditWarning") );
    $DiscussionForm->FatalError = 1;
    return;
    }
    }
    }
    // Add function todelegate
    $Context->AddToDelegate('DiscussionForm', 'PostLoadData', 'DiscussionForm_DiceRoller_EditValidation' );
    }

    Hope it helps
  10.  # 10
    Thanks so much! I can trust my students now! :-)
  11.  # 11
    You might want to let yourself view all whispers, too.

    p.s. Stop trying to stop kids cheating. That's half the fun of being at school!
    •  
      CommentAuthorWallPhone
    • CommentTimeSep 11th 2006
     # 12
    Thank you Arturo! With that ability removed from the users, I feel comfortable calling this version 1.0 now. It still feels a bit hacky, as the link and its list item are still in the source, but I will get those out sometime later.

    Fixed the edited => ed bug also.
    • CommentAuthorithcy
    • CommentTimeSep 11th 2006
     # 13
    @a_magical_me: ah, i always forget :)
  14.  # 14
    Nah, stopping kids from trying to cheat is half the fun of teaching... The other half is June, July, and August... :-)

    And how do you view all whispers?
  15.  # 15
    Its part of the role permissions.

1 to 15 of 15

  1.  
Add your comments
    Username Password
  • Format comments as
 
Back to Discussions Top of Page