Opened 14 years ago

Closed 14 years ago

#2139 closed enhancement (duplicate)

I wasn't easily able to tell what I had previously recorded (or had scheduled to record) in the canned searches

Reported by: myth@… Owned by: xris
Priority: trivial Milestone: unknown
Component: mythweb Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

I wasn't easily able to tell what movies were flagged as recorded, or were going to be recorded in future and this lead to a lot of times where I flagged the same show twice.

I came up with a quick and dirty patch against current SVN, not the ideal solution, but it does solve my problem.

Attachments (1)

search.patch (2.4 KB) - added by myth@… 14 years ago.
patch to indicate previously recorded, or up coming recordings for canned search

Download all attachments as: .zip

Change History (8)

Changed 14 years ago by myth@…

Attachment: search.patch added

patch to indicate previously recorded, or up coming recordings for canned search

comment:1 Changed 14 years ago by xris

Resolution: wontfix
Status: newclosed

Please separate code from interface (ie. no queries in the display code), and use the db handler referenced by all recent mythweb code.

comment:2 Changed 14 years ago by myth@…

Resolution: wontfix
Status: closedreopened

The code I post was an example of what could work, I didn't expect it to be taken verbatium. In any case closing this bug is pointless since there has been no solution commited.

comment:3 Changed 14 years ago by Stuart Auchterlonie

Resolution: wontfix
Status: reopenedclosed

You have been politely asked by the maintainer to rework your patch to make it suitable for inclusion in myth. If you wish to see it in myth, please do so.

comment:4 Changed 14 years ago by anonymous

Resolution: wontfix
Status: closedreopened

Let me rephrase, I'm not trying to get my patch into myth, I don't have the time/inclination to learn how to do things the right way.

I was suggesting a feature, not suggesting my patch be used, the only reason I uploaded what I had was a proof of concept of what I was hoping to see in myth.

If you guys persist in closing this bug, that's your perogative, but it tends to sour people from bothering to make suggestions in future.

comment:5 Changed 14 years ago by anonymous

Hello,

There are numerous places for making code suggestions - trac is for code that you wish to commit. Perhaps copying to the dev list or on IRC first. There is no intention to dissuade from suggestions - its just a bit annoying when Trac is used. You could also list it here:

http://www.mythtv.org/wiki/index.php/Feature_Wishlist

Regards Chris

comment:6 Changed 14 years ago by sphery <mtdean@…>

As asked in http://www.gossamer-threads.com/lists/mythtv/dev/217473#217473 , isn't the effect of the patch the same as that of mythweb-recstatus_without_subtitle_and_description.patch in #1874?

comment:7 Changed 14 years ago by xris

Resolution: duplicate
Status: reopenedclosed

If sphery is correct, I'll be applying his patch shortly. If he is not, leave this ticket closed unless there is good code, or create a new ticket that actually describes what's going wrong.

Note: See TracTickets for help on using tickets.