Forms not update in general manage panel

Bug #741744 reported by Quentin Estoppey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Eventum
Won't Fix
Low
Elan Ruusamäe

Bug Description

The 4 new options for default notifications options cannot be update through mange/general.php page.
Please ass a test (isset) for the 4 new checkbox:
$setup["update"] = isset($_POST["update"]) ? $_POST["update"] : 0;
$setup["closed"] = isset($_POST["closed"]) ? $_POST["closed"] : 0;
$setup["emails"] = isset($_POST["emails"]) ? $_POST["emails"] : 0;
$setup["files"] = isset($_POST["files"]) ? $_POST["files"] : 0;

Otherwise ther is an empty error

Revision history for this message
Quentin Estoppey (quentin-estoppey) wrote :

Again I remember now that file notification.tpl.html doen't take in account the setup of thise options. If you set not notifications options by defaults (checkboxes unchecked in general manage panel), then the web interface still have the box checked by default.

To take in account the option setup, code should be like that (add of $default_actions.emails==1 condition in if):
<input type="checkbox" name="actions[]" {if ($info == '' && $default_actions.emails == 1) || $info.emails == 1}checked{/if} value="emails"> <a id="link" class="link" href="javascript:void(null);" onClick="toggleCheckbox('notification_form', 'actions[]', 0);">{t}Emails are Received or Sent{/t}</a><br />
            <input type="checkbox" name="actions[]" {if ($info == '' && $default_actions.updated == 1) || $info.updated == 1}checked{/if} value="updated"> <a id="link" class="link" href="javascript:void(null);" onClick="toggleCheckbox('notification_form', 'actions[]', 1);">{t}Overview or Details are Changed{/t}</a><br />
            <input type="checkbox" name="actions[]" {if ($info == '' && $default_actions.closed == 1) || $info.closed == 1}checked{/if} value="closed"> <a id="link" class="link" href="javascript:void(null);" onClick="toggleCheckbox('notification_form', 'actions[]', 2);">{t}Issue is Closed{/t}</a><br />
            <input type="checkbox" name="actions[]" {if ($info == '' && $default_actions.files == 1) || $info.files == 1}checked{/if} value="files"> <a id="link" class="link" href="javascript:void(null);" onClick="toggleCheckbox('notification_form', 'actions[]', 3);">{t}Files are Attached{/t}</a>

Elan Ruusamäe (glen666)
Changed in eventum:
milestone: none → 2.3.4
Elan Ruusamäe (glen666)
Changed in eventum:
milestone: 2.3.4 → 2.4
Revision history for this message
Elan Ruusamäe (glen666) wrote :

seems to be working in 2.4.0-pre1

Changed in eventum:
assignee: nobody → Elan Ruusamäe (glen666)
importance: Undecided → Low
status: New → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.