Opened 14 years ago
Closed 14 years ago
#8383 closed patch (fixed)
mythweb: patch to delete channels
Reported by: | 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)
Change History (4)
Changed 14 years ago by
Attachment: | set_channels.php added |
---|
comment:1 Changed 14 years ago by
Milestone: | 0.23 → unknown |
---|---|
Priority: | minor → trivial |
Severity: | high → low |
Please see the ticket howto, do not set milestones/severity/priority.
comment:2 Changed 14 years ago by
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 ;-)
set_channels.php (whole file, no diff - sorry. lines 18-35 relevant)