Opened 16 years ago

Closed 15 years ago

#6510 closed defect (fixed)

extended search options apply only to advanced search, falsely display for others

Reported by: iskimj@… Owned by: xris
Priority: minor Milestone: 0.24
Component: Plugin - MythWeb Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

The additional search options (only match HD, only match commercial free, ignore generic shows) apply only to advanced searches (neither quick nor canned searched). The problem is that the search results page displayed after using quick or canned searches will show those options checked but the results do not reflect those options.

If the desire is to have these options only apply to advanced searches, then those settings should be cleared when performing a quick or canned search to avoid confusion.

Alternatively, these could be preferences that apply to all searches. I would prefer this behavior and have attached a simple patch for review; the patch moves the associated extra_query statements so that they are applied to all types of searches.

Attachments (1)

search-options.patch (1.5 KB ) - added by iskimj@… 16 years ago.
patch: modules/tv/search.php

Download all attachments as: .zip

Change History (4)

by iskimj@…, 16 years ago

Attachment: search-options.patch added

patch: modules/tv/search.php

comment:1 by Rob Smith, 16 years ago

Milestone: unknown0.23
Version: unknownhead

comment:2 by Rob Smith, 16 years ago

Milestone: 0.230.24

comment:3 by Rob Smith, 15 years ago

Resolution: fixed
Status: newclosed

(In [24936]) Fixes #6510, moves the extended search options to be applied globally rather then just to the advanced search

Note: See TracTickets for help on using tickets.