Modify

Opened 14 years ago

Closed 10 years ago

#7582 closed enhancement (fixed)

[patch] Filter result pages

Reported by: vaclav.cejka@… Owned by: Geert
Priority: normal Component: WantedPagesMacro
Severity: normal Keywords: patch
Cc: Itamar Oren, Geert Trac Release: 0.11

Description

Among really wanted pages, many other CamelCase strings either in original basic wiki, or i.e. function names or class names in ticket description appear as false wanted page. It might be helpful to specify regex as an parameter to filter only wanted page of some pattern, and thus eliminate like missing TracXXX etc. pages.

Attachments (1)

wantedpagesplugin-filter.patch (2.1 KB) - added by Benjamin Lau 13 years ago.
filter patch for WantedPagesPlugin

Download all attachments as: .zip

Change History (11)

comment:1 Changed 13 years ago by Benjamin Lau

Keywords: patch added
Summary: Filter result pages[patch] Filter result pages

I've created a patch for this. It adds a "filter=<filter>" argument to the macro. Here's an example:

[[WantedPages(show_referrers,filter=.*?Trac.*|Sandbox|.*?Wiki.*|Inter.*)]]

The pages matched by the filter are not displayed.

Changed 13 years ago by Benjamin Lau

filter patch for WantedPagesPlugin

comment:2 Changed 13 years ago by Itamar Oren

Cc: Itamar Oren added; anonymous removed

Thanks for the patch!

Just started using it, and looks like it does the job.

Any chance it gets into trunk?

comment:3 Changed 13 years ago by Benjamin Lau

Well... It looks like jfrancis (the current maintainer) hasn't been over here in a while. So someone would need to adopt the plugin and merge the patch. I wouldn't be against doing this myself... Just won't have time until after the new year.

comment:4 Changed 11 years ago by Ryan J Ollos

Owner: changed from Justin Francis to Ryan J Ollos
Status: newassigned

comment:5 Changed 11 years ago by Ryan J Ollos

Component: WantedPagesPluginWantedPagesMacro

Renaming plugin from WantedPagesPlugin to WantedPagesMacro.

comment:6 Changed 11 years ago by Ryan J Ollos

Cc: Geert added

#10819 closed as a duplicate, with the specific request to exclude the Trac documentation.

comment:7 in reply to:  3 Changed 11 years ago by Ryan J Ollos

Replying to netjunki:

Well... It looks like jfrancis (the current maintainer) hasn't been over here in a while. So someone would need to adopt the plugin and merge the patch. I wouldn't be against doing this myself... Just won't have time until after the new year.

See #11491.

comment:8 Changed 11 years ago by Ryan J Ollos

Owner: changed from Ryan J Ollos to Geert

Reassigning tickets to glinders, assuming adoption request in #11491 will be approved.

comment:9 Changed 10 years ago by Geert

Added ignoring referring pages using regex, based on the patch supplied above.

Updated Wiki page WantedPagesMacro to include the new option.

See revision r14076 for details.

comment:10 Changed 10 years ago by Geert

Resolution: fixed
Status: assignedclosed

Included patch in the macro, see comment:9

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Geert.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.