Opened 13 years ago

Closed 13 years ago

#2718 closed patch (fixed)

Add buttons to queue jobs on recorded programs

Reported by: jeff@… Owned by: xris
Priority: minor Milestone: unknown
Component: mythweb Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Adds buttons to each program's entry in the "Recorded Programs" listing, allowing a recorded program to be submitted for commercial flagging, transcoding, or any of the four user jobs that have been defined. User jobs are identified by their user-entered descriptions (i.e., they aren't translated).

Also moves the current "Delete" and "Delete + Rerecord" buttons into the "status" row for each entry, removing a column in the full table. The formatting of the various status indicators are slightly affected, converting the current <span> elements to table cells. This looks OK to me, but others may have a different aesthetic.

The new buttons, and the relocation of the existing buttons, occurs under the control of a $buttons_under variable, which is currently set "TRUE" directly in the code, but could alternatively be exposed as a user preference.

This patch has been tested against the 0.20 release. The svn sources don't patch directly from the same diff, due to the new "has been viewed" flag, but I believe I updated the diffs appropriately. However, I don't have a backend compatible with the current svn sources, however, so have not been able to verify this specific patch.

I'm not sure what the usual protocol is, but this patch doesn't include the results of re-running modules/_shared/lang/build_translation.pl.

Attachments (3)

job-queue.diff (16.7 KB) - added by jeff@… 13 years ago.
Patch for job queue buttons
screenshot_02.jpg (201.8 KB) - added by jeff@… 13 years ago.
Sample screen shot
jobqueue.php (322 bytes) - added by jeff@… 13 years ago.
Missing jobqueue.php file (oops)

Download all attachments as: .zip

Change History (6)

Changed 13 years ago by jeff@…

Attachment: job-queue.diff added

Patch for job queue buttons

Changed 13 years ago by jeff@…

Attachment: screenshot_02.jpg added

Sample screen shot

comment:1 Changed 13 years ago by anonymous

The required jobqueue.php isn't included in the diff.

Changed 13 years ago by jeff@…

Attachment: jobqueue.php added

Missing jobqueue.php file (oops)

comment:2 in reply to:  1 Changed 13 years ago by jeff@…

Replying to anonymous:

The required jobqueue.php isn't included in the diff.

Missing file uploaded now. Sorry for the trouble.

comment:3 Changed 13 years ago by xris

Resolution: fixed
Status: newclosed

(In [13002]) Add jobqueue status and queueing to the recording details page. The layout is a bit messy, but it will get fixed before release. This also happens to close #2718.

Note: See TracTickets for help on using tickets.