#59 Apply Content Filters on User Karma

closed
Chris Nandor
7
2002-11-19
2001-09-15
chromatic
No

This patch adds a 'karma' field to content filters. If
a karma level is specified, the filter will not be
applied to comments from users with karma above the level.

It modifies the content_filters database schema, as
well as the default content filters SQL statements. In
addition, it adds the karma field to the display and
edit templates for content filters.

The default behavior is not changed from current behavior.

Discussion

  • chromatic
    chromatic
    2001-09-15

    Comprehensive patch

     
    Attachments
  • Chris Nandor
    Chris Nandor
    2001-10-04

    Logged In: YES
    user_id=3660

     
  • Chris Nandor
    Chris Nandor
    2001-10-04

    • assigned_to: nobody --> capttofu
     
  • Jamie McCarthy
    Jamie McCarthy
    2001-11-02

    Logged In: YES
    user_id=3889

    CaptTofu, I don't believe this would be a helpful change to the code. Unless CmdrTaco disagrees, I suggest you just close this bug.

     
  • Chris Nandor
    Chris Nandor
    2001-11-02

    Logged In: YES
    user_id=3660

    I don't care much one way or the other. Note that the code
    should be switchable via a var. I've not looked at the
    patch, except in the text of the book, where it was not
    switchable via a var.

     
  • chromatic
    chromatic
    2001-11-02

    Logged In: YES
    user_id=2274

    Shane @ lottadot suggested a per-user variable
    (skipfilters). The other option is a configuration variable
    (karmafilters?). I'd be happy to add in either.

    Note that the default value of the karma field on a filter
    is zero, which preserves the existing behavior (that is,
    user karma has no effect).

     
  • Jamie McCarthy
    Jamie McCarthy
    2001-11-02

    Logged In: YES
    user_id=3889

    I'm going to seriously consider this over the weekend and talk to Rob about it -- the more I think about it the more I like the idea -- but it doesn't look like it'll make it into 2.2.0.

     
  • Jamie McCarthy
    Jamie McCarthy
    2001-11-02

    • assigned_to: capttofu --> jamiemccarthy
     
  • Jamie McCarthy
    Jamie McCarthy
    2002-09-03

    • priority: 5 --> 7
     
  • Chris Nandor
    Chris Nandor
    2002-11-19

    • assigned_to: jamiemccarthy --> pudge
    • status: open --> closed
     
  • Chris Nandor
    Chris Nandor
    2002-11-19

    Logged In: YES
    user_id=3660

    This is so freaking old that it likely doesn't apply to the
    current codebase. I am going to close it. If you still
    want it, please update the patch (though you might want to
    check with me first to make sure we will accept it [I'd need
    to talk to Rob] so you don't waste your time on it).