PacketFence - BTS - PacketFence
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0001071PacketFenceweb adminpublic2010-09-17 12:042012-02-28 14:31
Reporterobilodeau 
Assigned Toobilodeau 
PrioritylowSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version3.2.0Fixed in Version3.2.0 
fixed in git revision
fixed in mtn revisionba0bb820bb367701791cf47bb5e6c81202a157d2
Summary0001071: most lists in the web admin don't remember your position bringing you back to page 1 on deletion
DescriptionTo reproduce:

- Open violation page
- Go to page X (where X > 1)
- delete a node

Result: brought back to page 1

Expected result: stay where you were

Node screen is the only one not affected because it uses the smarter paging mechanisms which relies on LIMITs SQL calls.

More an annoyance than anything..
Steps To Reproduce
Additional Information
Tagsusability
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2010-09-17 12:04obilodeauNew Issue
2010-09-17 12:04obilodeauTag Attached: usability
2010-11-19 14:25obilodeauTarget Version1.10.0 => 2.0.0
2011-01-18 09:47obilodeauTarget Version2.0.0 => 2.1.0
2011-03-03 15:15obilodeauTarget Version2.1.0 => +1
2011-03-03 15:18obilodeauTarget Version+1 => +2
2012-01-03 11:27obilodeauStatusnew => assigned
2012-01-03 11:27obilodeauAssigned To => obilodeau
2012-01-03 11:42obilodeauNote Added: 0002509
2012-01-03 11:55obilodeaumtn revision => ba0bb820bb367701791cf47bb5e6c81202a157d2
2012-01-03 11:55obilodeauNote Added: 0002510
2012-01-03 11:55obilodeauStatusassigned => resolved
2012-01-03 11:55obilodeauFixed in Version => trunk
2012-01-03 11:55obilodeauResolutionopen => fixed
2012-02-28 14:22obilodeauTarget Version+2 => 3.2.0
2012-02-28 14:22obilodeauFixed in Versiontrunk => 3.2.0
2012-02-28 14:31obilodeauNote Added: 0002594
2012-02-28 14:31obilodeauStatusresolved => closed

Notes
(0002509)
obilodeau   
2012-01-03 11:42   
Reason for the node screen being unaffected is not because of the SQL but because of the header.php include performs a table refresh and the set_linkable and the set_page_num and set_per_page calls on the table are done _after_ the refresh unlike violation and others.

Several pages were well done in the first place.
(0002510)
obilodeau   
2012-01-03 11:55   
Turns out the header include must be done earliest before table changes to avoid edit or delete side effects... Fixed every caller so hopefully this won't re-happen in the future.

Also fixed an issue with violation page loses violation vid description field.
(0002594)
obilodeau   
2012-02-28 14:31   
Fixed in recently released 3.2.0.