SiT! Bugs

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001806SiT!incidentspublic2012-10-02 20:412012-10-16 21:57
Reporterpaulh 
Assigned To 
PrioritynormalSeverityminorReproducibilityalways
StatusnewResolutionopen 
PlatformOSOS Version
Product VersionCurrent GIT 
Target VersionFixed in Version 
Summary0001806: create_incident() in incident.inc.php doesn't do triggers
DescriptionThis function doesn't initiate any of the incident creation triggers

applies to git and svn
TagsNo tags attached.
Attached Files

- Relationships

-  Notes
User avatar (0004532)
Tomse (developer)
2012-10-02 20:46
edited on: 2012-10-02 20:48

which trigger should be there?

triggers are located after the incident_create() is called, in the pages..
i.e. portal/add.php
or am I wrong ?

User avatar (0004533)
Tomse (developer)
2012-10-06 09:17

I read wrong, sorry. I read it as being inside the function. and not after the call.
User avatar (0004550)
paulh (administrator)
2012-10-16 21:14

I'm in too minds now, the incident_create() function doesn't trigger any actions so you have to code the trigger afterwards which leaves this upto third party plugin developers who may not know this (I didn't ;-))

But I can also see a case when you may not want to trigger, I'm torn now as to what the correct functionality should be....
User avatar (0004551)
nicdev (developer)
2012-10-16 21:57

That just sparked something. I remember having to add some triggers to our auto-create plugin, to make it work. I think I did speak to Ivan about it (yonks ago), but it was easy to add tot he plugin, so we left it at that ...

- Issue History
Date Modified Username Field Change
2012-10-02 20:41 paulh New Issue
2012-10-02 20:46 Tomse Note Added: 0004532
2012-10-02 20:48 Tomse Note Edited: 0004532 View Revisions
2012-10-06 09:17 Tomse Note Added: 0004533
2012-10-16 21:14 paulh Note Added: 0004550
2012-10-16 21:57 nicdev Note Added: 0004551


Copyright © 2000 - 2019 MantisBT Team
Powered by Mantis Bugtracker