1. This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn More.

Duplicate Spam checker not working on profile posts

Discussion in 'Resolved Bug Reports' started by Jon W, Sep 21, 2013.

  1. Jon W

    Jon W Well-Known Member

    In XenForo_ControllerPublic_Member, the actionSave controller checks for the value of discussion_state instead of message_state. As such, the spam checker never runs and all spam gets through.

    To fix, change:
    PHP:
                if (!$writer->hasErrors()
                    && 
    $writer->get('discussion_state') == 'visible'
                    
    && $spamModel->visitorRequiresSpamCheck()
                )
                {
    to:
    PHP:
                if (!$writer->hasErrors()
                    && 
    $writer->get('message_state') == 'visible'
                    
    && $spamModel->visitorRequiresSpamCheck()
                )
                {
     
    Marcus likes this.
  2. Mike

    Mike XenForo Developer Staff Member

  3. Jon W

    Jon W Well-Known Member

    Thanks.
     
  4. Marcus

    Marcus Well-Known Member

    Nice find, I queried the message_state for threads some time ago :)
     

Share This Page