Anonymous | Login | Signup for a new account | 2021-03-08 02:42 GMT | ![]() |
Main | My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0001592 | SiT! | triggers | public | 2011-06-14 20:16 | 2012-01-05 12:39 | ||||
Reporter | paulh | ||||||||
Assigned To | paulh | ||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||
Status | resolved | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | Current GIT | ||||||||
Target Version | 3.90beta1 | Fixed in Version | Current GIT | ||||||
Summary | 0001592: action_details.php produces javascript errors and save trigger | ||||||||
Description | When adding your own Notifications you get JS errors about unknown objects and when you save the notification the my notifications page doesn't list it | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | |||||||||
![]() |
|
Tomse (developer) 2011-06-14 20:29 |
I did 2 tests.. Notify when new KB article and Mail when Incident nearing SLA none of these gave any errors, and the "triggers" are displayed on my page (I did these triggers with i18n da-DK) |
paulh (administrator) 2011-06-14 20:45 |
You get the JS error when adding a notification for "When a new contract is added" my mistake on the second part I'd failed to tick "Enable condition" I didn't spot this before, I though seen as I'd defined the condition it would be enabled (perhaps we need to tweak the interface?) |
ivan (administrator) 2011-06-14 20:50 |
yeah I think users would expect a new condition to be enabled by default |
ivan (administrator) 2011-06-14 21:01 |
in fact ... it's not possible to edit the conditions/checks/rules on a notification as far as I can tell so the checkbox to enable/disable seems redundant to me. Maybe this was intended for future use? |
ivan (administrator) 2011-06-24 18:06 |
I've fixed the problem with the notifications recently added not being shown, it was happening because the notification user was hardcoded to 1 (admin) so only that user could add notifications and have them displayed. Other users just added notifications for admins list. |
ivan (administrator) 2011-08-21 14:18 |
I can't reproduce the JS error mentioned in this report, is it still an issue? Can you post steps? |
Tomse (developer) 2011-12-03 17:32 |
I've been trying to reproduce this, but I'm not able to |
paulh (administrator) 2011-12-20 16:10 |
I've just retested and can reproduce with teh latest git * Login as a user (I was using admin) * Go to My Notifications (notifications.php) * Click "New Notification" * For action select "When a new contract is added" you get Error: $("parametersbox") is null Source File: http://localhost/sit4/scripts/webtrack.js [^] Line: 1367 In the JS debugger |
paulh (administrator) 2011-12-20 16:15 |
looks like this was removed from SVN in r2240 |
paulh (administrator) 2011-12-20 16:23 |
f1f1a33 removes a number of fields which the JS was hiding which no longer exist in 4.x |
![]() |
|||
Date Modified | Username | Field | Change |
2011-06-14 20:16 | paulh | New Issue | |
2011-06-14 20:29 | Tomse | Note Added: 0003972 | |
2011-06-14 20:30 | Tomse | Assigned To | => Tomse |
2011-06-14 20:30 | Tomse | Status | new => feedback |
2011-06-14 20:45 | paulh | Note Added: 0003974 | |
2011-06-14 20:45 | paulh | Status | feedback => assigned |
2011-06-14 20:45 | paulh | Assigned To | Tomse => |
2011-06-14 20:45 | paulh | Status | assigned => new |
2011-06-14 20:50 | ivan | Note Added: 0003977 | |
2011-06-14 21:01 | ivan | Note Added: 0003979 | |
2011-06-14 21:02 | ivan | Status | new => confirmed |
2011-06-14 21:02 | ivan | Target Version | => 3.90beta1 |
2011-06-24 18:06 | ivan | Note Added: 0004068 | |
2011-08-21 14:18 | ivan | Note Added: 0004251 | |
2011-08-21 14:18 | ivan | Status | confirmed => feedback |
2011-12-03 17:32 | Tomse | Note Added: 0004351 | |
2011-12-20 16:10 | paulh | Note Added: 0004384 | |
2011-12-20 16:10 | paulh | Status | feedback => new |
2011-12-20 16:10 | paulh | Status | new => acknowledged |
2011-12-20 16:15 | paulh | Note Added: 0004385 | |
2011-12-20 16:23 | paulh | Note Added: 0004386 | |
2011-12-20 16:23 | paulh | Assigned To | => paulh |
2011-12-20 16:23 | paulh | Status | acknowledged => resolved |
2011-12-20 16:23 | paulh | Resolution | open => fixed |
2011-12-20 16:23 | paulh | Fixed in Version | => Current GIT |
Copyright © 2000 - 2021 MantisBT Team |