Opened 10 years ago

Closed 9 years ago

#8383 closed patch (fixed)

mythweb: patch to delete channels

Reported by: joliver@… Owned by: Rob Smith
Priority: trivial Milestone: unknown
Component: Plugin - MythWeb Version: 0.23rc2
Severity: low Keywords:
Cc: Ticket locked: no

Description

There are two reasons why deleting channels in 0.23rc2 doesn't work:

1) if the delete-checkbox is placed before the channum field in the layout, $chanid isn't set (or the the wrong $chanid is set!). $chanid has to be determined from its own checkbox name.

2) unfornunately the popular suhosin security patch for php (default in debian) restricts the maximal array size of post to 64. The channels form usually has much more fields, the "save" button is the last field and never arrives the script. Result: no values are stored. http://www.hardened-php.net/suhosin/configuration.html#suhosin.post.max_array_depth

Attachments (1)

set_channels.php (4.5 KB) - added by joliver@… 10 years ago.
set_channels.php (whole file, no diff - sorry. lines 18-35 relevant)

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by joliver@…

Attachment: set_channels.php added

set_channels.php (whole file, no diff - sorry. lines 18-35 relevant)

comment:1 Changed 10 years ago by robertm

Milestone: 0.23unknown
Priority: minortrivial
Severity: highlow

Please see the ticket howto, do not set milestones/severity/priority.

comment:2 Changed 10 years ago by joliver@…

Update 2) the relevant parameter ist "suhosin.post.max_vars" (default:200). A warning in the documentation and in the form (online/html) would be nice. Changing the form to ajax would be a nice solution but a little bit more work ;-)

comment:3 Changed 9 years ago by Rob Smith

Resolution: fixed
Status: newclosed

(In [24934]) Fixes #8383, this fixes an issue where the page layout can affect the channel being deleted

Note: See TracTickets for help on using tickets.