Bug 22516

Summary: accountlines.lastincrement can be removed
Product: Koha Reporter: Martin Renvoize <martin.renvoize>
Component: Architecture, internals, and plumbingAssignee: Martin Renvoize <martin.renvoize>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: enhancement    
Priority: P5 - low CC: josef.moravec, kyle, m.de.rooy, martin.renvoize, nick, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 14591, 21683    
Bug Blocks: 14825    
Attachments: Bug 22516: Remove remaining calls to lastincrement
Bug 22516: Database update
Bug 22516: Remove remaining calls to lastincrement
Bug 22516: Database update
Bug 22516: Remove remaining calls to lastincrement
Bug 22516: Database update
Bug 22516: Remove remaining calls to lastincrement
Bug 22516: Database update

Description Martin Renvoize 2019-03-14 17:07:53 UTC
Bug 14591 will remove the last use case for 'lastincrement', we should remove this field from the database once that bug is pushed.
Comment 1 Martin Renvoize 2019-03-15 08:43:55 UTC
Created attachment 86637 [details] [review]
Bug 22516: Remove remaining calls to lastincrement

A few places still set lastincrement and the tests still checked for
correct values. But nowhere in the code still uses lastincrement for
anything meaningful.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 2 Martin Renvoize 2019-03-15 08:43:58 UTC
Created attachment 86638 [details] [review]
Bug 22516: Database update

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 3 Martin Renvoize 2019-03-15 08:45:19 UTC
Test plan.. in reality this shouldn't affect anything as the real testing of functionality will have happened in bug 14591. So.. test plan is to simply run the affected tests and make sure they are still passing.
Comment 4 Martin Renvoize 2019-03-15 10:35:28 UTC
Created attachment 86662 [details] [review]
Bug 22516: Remove remaining calls to lastincrement

A few places still set lastincrement and the tests still checked for
correct values. But nowhere in the code still uses lastincrement for
anything meaningful.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 5 Martin Renvoize 2019-03-15 10:35:31 UTC
Created attachment 86663 [details] [review]
Bug 22516: Database update

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 6 Kyle M Hall 2019-03-18 12:43:17 UTC
Created attachment 86712 [details] [review]
Bug 22516: Remove remaining calls to lastincrement

A few places still set lastincrement and the tests still checked for
correct values. But nowhere in the code still uses lastincrement for
anything meaningful.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 7 Kyle M Hall 2019-03-18 12:43:26 UTC
Created attachment 86713 [details] [review]
Bug 22516: Database update

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 8 Marcel de Rooy 2019-03-22 11:26:17 UTC
Created attachment 86888 [details] [review]
Bug 22516: Remove remaining calls to lastincrement

A few places still set lastincrement and the tests still checked for
correct values. But nowhere in the code still uses lastincrement for
anything meaningful.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 9 Marcel de Rooy 2019-03-22 11:26:21 UTC
Created attachment 86889 [details] [review]
Bug 22516: Database update

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 10 Marcel de Rooy 2019-03-22 11:27:22 UTC
@RM: DBIx update !
Comment 11 Nick Clemens 2019-03-28 18:19:42 UTC
Awesome work all!

Pushed to master for 19.05
Comment 12 Martin Renvoize 2019-04-01 13:58:21 UTC
Enhancement, will not be backported to 18.11.x series.