I needed to hide some filters like uid, so client won't see them.
Here is the patch which will add checkboxes to display filters to settings page.

I also noticed that there is a bug with displaying status, so it is fixed in this patch aswell

CommentFileSizeAuthor
#2 advuser.module_0.patch3.87 KBsign
advuser_0.patch4.05 KBsign

Comments

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Needs work

This is an interesting patch. Let me ponder it more.

Can you please supply a separate patch for the displaying status fix. Submit it with a new bug report that is properly titled. I've marked "code needs work" because you'll need to re-roll this patch to remove the displaying status piece.

sign’s picture

sign commented
StatusFileSize
new advuser.module_0.patch3.87 KB

Here it is

sign’s picture

sign commented
Status: Needs work » Needs review
Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Needs work

@Sign: You set +define('ADVUSER_DEFAULT_FILTERS', NULL); but never used it. I think you meant to use it in + $values = variable_get('advuser_filters_check', ''); didn't you?

Anonymous’s picture

Anonymous (not verified) commented
Assigned: sign »
Anonymous’s picture

Anonymous (not verified) commented
Version: 5.x-1.x-dev » 5.x-2.x-dev
Status: Needs work » Postponed
Anonymous’s picture

Anonymous (not verified) commented
Title: Hide filters option » Selectable filters settings
Version: 5.x-2.x-dev » 6.x-3.x-dev
Anonymous’s picture

Anonymous (not verified) commented
Title: Selectable filters settings » Optional filter fields
Status: Postponed » Active

I will make this happen but the default_values will be set to true for all of the fields.

Anonymous’s picture

Anonymous (not verified) commented
Version: 6.x-3.x-dev » 7.x-3.x-dev
Status: Active » Postponed

Pushing feature request to 7.x version.