[patch] Activity Stream filtering $hidden_sql

[DJPaul]

Please can this patch get into trunk for 1.3. I have previously talked to JJJ about this. It will let me use the 'hide_sitewide' column in the activity stream database to mark entries as spam, allowing integration into Akismet (which I'm going to provide a patch for).

[DJPaul]

Is using an "IN" like this slower than "(a.hide_sitewide = 0 OR a.hide_sitewide = 1)"? What's best?

[DJPaul]

Boone, what are your thoughts on this? The idea was to use this column to allow plugins marking activity entries as spam. For some components which may get custom post type'd, we'd get this "for free", but I think activity streams is likely to not be custom post type'd and therefore has a slightly better justification for this sort of thing. I'm reluctant to add a is_spam column just for this reason, however.

[boonebgorges]

The general strategy is fine with me. How about testing hide_sitewide > 0 so that plugins could extend this field even further?

[DJPaul]

It's a better idea to sort out the duplicate uses of hide_sitewide without adding more confusion to the mix. For the interim, I'll use meta or a JOIN.