PacketFence - BTS - PacketFence
View Issue Details
0000917PacketFenceerror-handlingpublic2010-02-24 12:492015-02-13 15:26
obilodeau 
obilodeau 
normalminoralways
closedopen 
 
 
0000917: node add fails on key constraint and they are not validated beforehand
DBI error string
Cannot add or update a child row: a foreign key constraint fails (`pf/node`, CONSTRAINT `0_57` FOREIGN KEY (`pid`) REFERENCES `person` (`pid`) ON DELETE CASCADE ON UPDATE CASCADE). (errno: 1452)

We should avoid that: either auto-create the person or bail appropriately.

Currently both Web UI and pfcmd call fails silently.
usability
duplicate of 0000840closed obilodeau pfcmd node edit (web node edit) fails silently 
related to 0001311closed  person and identifier are confusing concepts 
Issue History
2010-02-24 12:49obilodeauNew Issue
2010-02-24 12:49obilodeauStatusnew => assigned
2010-02-24 12:49obilodeauAssigned To => obilodeau
2010-05-05 13:46obilodeauCategory1.8.7 => error-handling
2010-05-05 13:46obilodeauTarget Version => 1.9.1
2010-09-15 13:20obilodeauTag Attached: usability
2010-09-15 13:20obilodeauTarget Version1.9.1 => 1.9.2
2010-09-22 16:02obilodeauTarget Version1.9.2 => 1.9.3
2011-02-15 08:04hawkNote Added: 0001862
2011-02-15 09:47obilodeauNote Added: 0001863
2011-02-15 09:47obilodeauTarget Version1.9.3 => 2.0.2
2011-03-03 15:19obilodeauTarget Version2.0.2 => +1
2011-10-17 10:01obilodeauRelationship addedduplicate of 0000840
2011-10-17 10:15obilodeauNote Added: 0002361
2011-10-17 10:19obilodeauRelationship addedrelated to 0001311
2015-02-13 15:26lmunroNote Added: 0003724
2015-02-13 15:26lmunroStatusassigned => closed

Notes
(0001862)
hawk   
2011-02-15 08:04   
Think I have also hit this bug!

If I "add" a "node" via the Web UI, but forget to add a "identifier" or uses an non-existing identifier, then the node will not get added,
BUT the Web UI reports "Added Record", which is false!

It was also not intuitive, to me, that the "identifier" refers to a "person" identifier.

New users playing with the admin interface is going to be really confused. I think fixing this bug, should have a higher priority.
(0001863)
obilodeau   
2011-02-15 09:47   
Rescheduled to 2.0.2. Thanks for the identifier remark.
(0002361)
obilodeau   
2011-10-17 10:15   
in node add, changed Identifier to Person Identifier as was mentioned
(0003724)
lmunro   
2015-02-13 15:26   
Old issues.
Most are not relevant to PF 4 and up.

Let's reopen the ones that matter when we move to github.