[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #148 from Kyle M Hall  ---
(In reply to Marcel de Rooy from comment #146)
> Created attachment 68383 [details] [review]
> Bug 14826: (QA Follow-up) Remove obsolete DBIx schema file
> 
> Removes Accountoffset.pm.
> Has been replaced by AccountOffset.pm on bug 14826.
> 
> Test plan:
> Run TestBuilder.t. Should pass now again.
> 
> Signed-off-by: Marcel de Rooy 

Thanks for the followup Marcel!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #147 from Kyle M Hall  ---
(In reply to Jonathan Druart from comment #145)
> (In reply to Jonathan Druart from comment #144)
> > Created attachment 68334 [details] [review] [review]
> > Bug 14826: account_offset_types.sql is not in the mandatory subdir
> 
> Pushed to master

Squee! Thanks Jonathan!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #146 from Marcel de Rooy  ---
Created attachment 68383
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68383&action=edit
Bug 14826: (QA Follow-up) Remove obsolete DBIx schema file

Removes Accountoffset.pm.
Has been replaced by AccountOffset.pm on bug 14826.

Test plan:
Run TestBuilder.t. Should pass now again.

Signed-off-by: Marcel de Rooy 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #145 from Jonathan Druart  
---
(In reply to Jonathan Druart from comment #144)
> Created attachment 68334 [details] [review]
> Bug 14826: account_offset_types.sql is not in the mandatory subdir

Pushed to master

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #144 from Jonathan Druart  
---
Created attachment 68334
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68334&action=edit
Bug 14826: account_offset_types.sql is not in the mandatory subdir

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Jonathan Druart  changed:

   What|Removed |Added

 Status|Passed QA   |Pushed to Master

--- Comment #143 from Jonathan Druart  
---
Pushed to master for 17.11, thanks to everybody involved!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-13 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Kyle M Hall  changed:

   What|Removed |Added

 Blocks||12768


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12768
[Bug 12768] Replacement cost and processing fee management
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #142 from Kyle M Hall  ---
Created attachment 68032
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68032&action=edit
Bug 14826 [QA Followup] - Make db update idempotent

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #141 from Kyle M Hall  ---
Created attachment 68031
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68031&action=edit
Bug 14826 [QA Followup] - Correct number of tests run following rebase

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #138 from Kyle M Hall  ---
Created attachment 68028
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68028&action=edit
Bug 14826: (QA followup) Only use plural modules in other modules

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #140 from Kyle M Hall  ---
Created attachment 68030
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68030&action=edit
Bug 14826: (QA followup) POD fixes

This patch just fixes POD. There are still some false positives with the
QA scripts.

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #139 from Kyle M Hall  ---
Created attachment 68029
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68029&action=edit
Bug 14826: Add account offset type table

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #136 from Kyle M Hall  ---
Created attachment 68026
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68026&action=edit
Bug 14826: Add unit tests for _FixAccountForLostAndReturned and
_FixOverduesOnReturn

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #137 from Kyle M Hall  ---
Created attachment 68027
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68027&action=edit
Bug 14826: (QA followup) Fix minor issues with unit tests

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #135 from Kyle M Hall  ---
Created attachment 68025
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68025&action=edit
Bug 14826: Add offset for rental fees

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #134 from Kyle M Hall  ---
Created attachment 68024
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68024&action=edit
Bug 14826: Add new offset table schema file

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #133 from Kyle M Hall  ---
Created attachment 68023
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68023&action=edit
Bug 14826: Remove old offset table schema file

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #132 from Kyle M Hall  ---
Created attachment 68022
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68022&action=edit
Bug 14826: Remove old offset table schema file

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #131 from Kyle M Hall  ---
Created attachment 68021
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68021&action=edit
Bug 14826; Unit Tests

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #129 from Kyle M Hall  ---
Created attachment 68019
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68019&action=edit
Bug 14826: Add Koha::Account::Offset(s) modules

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Kyle M Hall  changed:

   What|Removed |Added

 Status|Failed QA   |Passed QA

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Kyle M Hall  changed:

   What|Removed |Added

  Attachment #67936|0   |1
is obsolete||
  Attachment #67937|0   |1
is obsolete||
  Attachment #67938|0   |1
is obsolete||
  Attachment #67939|0   |1
is obsolete||
  Attachment #67940|0   |1
is obsolete||
  Attachment #67942|0   |1
is obsolete||
  Attachment #67943|0   |1
is obsolete||
  Attachment #67944|0   |1
is obsolete||
  Attachment #67945|0   |1
is obsolete||
  Attachment #67946|0   |1
is obsolete||
  Attachment #67947|0   |1
is obsolete||
  Attachment #67948|0   |1
is obsolete||
  Attachment #67949|0   |1
is obsolete||

--- Comment #128 from Kyle M Hall  ---
Created attachment 68018
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68018&action=edit
Bug 14826: Update database

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #130 from Kyle M Hall  ---
Created attachment 68020
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68020&action=edit
Bug 14826: Resurrect account offsets table

The account offsets table should be used to track increments and
decrements of fines via payments and credits, as well as fine accruals.
It should be able to match fees to payments and visa versa, so we can
know which fee was paid by a given payment, and which payments applied
to a given fee.

Test Plan:
1) Apply this patch
2) Run updatedatabase
3) Note the table accountoffsets has been renamed to account_offsets
4) Ensure fine generation creates offsets
5) Ensure creating a manual invoice creates an offset
6) Ensure a lost item charge creates an offset
7) Ensure Reverse Payment creates an offset
8) Ensure a payment creates an offset
9) Ensure a payment for multiple fees creates an offset for each
10) Ensure writeoffs create offsets

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Kyle M Hall  changed:

   What|Removed |Added

  Attachment #67941|0   |1
is obsolete||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Kyle M Hall  changed:

   What|Removed |Added

 Blocks|12768   |
  Text to go in the||The account offsets table
  release notes||allows libraries to know
   ||the entire history of fees
   ||and payments in Koha.
   ||Previously there was no way
   ||to directly connect fees
   ||and payments. The addition
   ||of the account offsets
   ||table allows you to know
   ||which fees paid which fines
   ||visa versa. This data will
   ||be accessible via reports
   ||and later via the staff
   ||intranet ( via Bug 2696 ).


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12768
[Bug 12768] Replacement cost and processing fee management
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-12 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Kyle M Hall  changed:

   What|Removed |Added

 Blocks||12768


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12768
[Bug 12768] Replacement cost and processing fee management
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Jonathan Druart  changed:

   What|Removed |Added

 Status|Passed QA   |Failed QA

--- Comment #127 from Jonathan Druart  
---
1. I think these patches must be based on top of bug 12768 (or the reverse).

2. Could you document this enhancement for the release notes?

3.
(In reply to Jonathan Druart from comment #40)
> What would be a test plan for the 2 calls to _FixAccountForLostAndReturned
> in AddIssue and AddReturn?

?

4.

diff --git a/Koha/Schema/Result/Rating.pm b/Koha/Schema/Result/Rating.pm
deleted file mode 100644

Hum!?

At this point it may be better to remove these schema files from the patches, I
will regenerate them. Just make sure that the update DB process will generate
the same schema as new installations (accountoffsets is not dropped by the
atomic update for instance).

5. The update DB process fails if executed twice

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #126 from Kyle M Hall  ---
Created attachment 67949
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67949&action=edit
Bug 14826 [QA Followup] - Correct number of tests run following rebase

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #124 from Kyle M Hall  ---
Created attachment 67947
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67947&action=edit
Bug 14826: Add account offset type table

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #123 from Kyle M Hall  ---
Created attachment 67946
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67946&action=edit
Bug 14826: (QA followup) Only use plural modules in other modules

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #125 from Kyle M Hall  ---
Created attachment 67948
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67948&action=edit
Bug 14826: (QA followup) POD fixes

This patch just fixes POD. There are still some false positives with the
QA scripts.

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #122 from Kyle M Hall  ---
Created attachment 67945
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67945&action=edit
Bug 14826: (QA followup) Fix minor issues with unit tests

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #121 from Kyle M Hall  ---
Created attachment 67944
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67944&action=edit
Bug 14826: Add unit tests for _FixAccountForLostAndReturned and
_FixOverduesOnReturn

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #120 from Kyle M Hall  ---
Created attachment 67943
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67943&action=edit
Bug 14826: Add offset for rental fees

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #119 from Kyle M Hall  ---
Created attachment 67942
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67942&action=edit
Bug 14826: Add new offset table schema file

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #117 from Kyle M Hall  ---
Created attachment 67940
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67940&action=edit
Bug 14826: Remove old offset table schema file

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #118 from Kyle M Hall  ---
Created attachment 67941
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67941&action=edit
Bug 14826: Remove old offset table schema file

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Kyle M Hall  changed:

   What|Removed |Added

  Attachment #67453|0   |1
is obsolete||
  Attachment #67454|0   |1
is obsolete||
  Attachment #67455|0   |1
is obsolete||
  Attachment #67456|0   |1
is obsolete||
  Attachment #67457|0   |1
is obsolete||
  Attachment #67458|0   |1
is obsolete||
  Attachment #67459|0   |1
is obsolete||
  Attachment #67460|0   |1
is obsolete||
  Attachment #67461|0   |1
is obsolete||
  Attachment #67462|0   |1
is obsolete||
  Attachment #67463|0   |1
is obsolete||
  Attachment #67464|0   |1
is obsolete||
  Attachment #67465|0   |1
is obsolete||

--- Comment #113 from Kyle M Hall  ---
Created attachment 67936
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67936&action=edit
Bug 14826: Update database

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #115 from Kyle M Hall  ---
Created attachment 67938
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67938&action=edit
Bug 14826: Resurrect account offsets table

The account offsets table should be used to track increments and
decrements of fines via payments and credits, as well as fine accruals.
It should be able to match fees to payments and visa versa, so we can
know which fee was paid by a given payment, and which payments applied
to a given fee.

Test Plan:
1) Apply this patch
2) Run updatedatabase
3) Note the table accountoffsets has been renamed to account_offsets
4) Ensure fine generation creates offsets
5) Ensure creating a manual invoice creates an offset
6) Ensure a lost item charge creates an offset
7) Ensure Reverse Payment creates an offset
8) Ensure a payment creates an offset
9) Ensure a payment for multiple fees creates an offset for each
10) Ensure writeoffs create offsets

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #116 from Kyle M Hall  ---
Created attachment 67939
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67939&action=edit
Bug 14826; Unit Tests

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #114 from Kyle M Hall  ---
Created attachment 67937
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67937&action=edit
Bug 14826: Add Koha::Account::Offset(s) modules

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Kyle M Hall  changed:

   What|Removed |Added

 Status|Patch doesn't apply |Passed QA

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Jonathan Druart  changed:

   What|Removed |Added

 Status|Passed QA   |Patch doesn't apply

--- Comment #112 from Jonathan Druart  
---
Please rebase

Applying: Bug 14826: Add unit tests for _FixAccountForLostAndReturned and
_FixOverduesOnReturn
error: sha1 information is lacking or useless (t/db_dependent/Circulation.t).

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Katrin Fischer  changed:

   What|Removed |Added

 CC||katrin.fisc...@bsz-bw.de

--- Comment #111 from Katrin Fischer  ---
I share Martha's excitement :) We've been trying to fix Accounts for several
years and in several ways now. I hope this will move us further as we have been
experiencing more and more problems.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-06 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #110 from Martha Fuerst  ---
RM: Really excited for this patch set, given the dependencies. Do we know what
the plan is? Thanks!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-06 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #109 from Martha Fuerst  ---
RM: Really excited for this patch set, given the dependencies. Do we know what
the plan is? Thanks!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-10-06 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Marcel de Rooy  changed:

   What|Removed |Added

 CC||m.de.r...@rijksmuseum.nl

--- Comment #108 from Marcel de Rooy  ---
RM: Whats the plan with this patch set? Some reports in the SO queue depend on
it.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #107 from Tomás Cohen Arazi  ---
@RM: Please note that I obsoleted the (non-applying) schema file changes. They
need to be generated when pushing.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #106 from Tomás Cohen Arazi  ---
Patches rebased.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #105 from Tomás Cohen Arazi  ---
Created attachment 67465
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67465&action=edit
Bug 14826: (QA followup) POD fixes

This patch just fixes POD. There are still some false positives with the
QA scripts.

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #103 from Tomás Cohen Arazi  ---
Created attachment 67463
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67463&action=edit
Bug 14826: (QA followup) Only use plural modules in other modules

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #100 from Tomás Cohen Arazi  ---
Created attachment 67460
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67460&action=edit
Bug 14826: Add offset for rental fees

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #101 from Tomás Cohen Arazi  ---
Created attachment 67461
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67461&action=edit
Bug 14826: Add unit tests for _FixAccountForLostAndReturned and
_FixOverduesOnReturn

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #102 from Tomás Cohen Arazi  ---
Created attachment 67462
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67462&action=edit
Bug 14826: (QA followup) Fix minor issues with unit tests

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #104 from Tomás Cohen Arazi  ---
Created attachment 67464
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67464&action=edit
Bug 14826: Add account offset type table

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #99 from Tomás Cohen Arazi  ---
Created attachment 67459
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67459&action=edit
Bug 14826: Add new offset table schema file

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #98 from Tomás Cohen Arazi  ---
Created attachment 67458
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67458&action=edit
Bug 14826: Remove old offset table schema file

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #97 from Tomás Cohen Arazi  ---
Created attachment 67457
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67457&action=edit
Bug 14826: Remove old offset table schema file

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #96 from Tomás Cohen Arazi  ---
Created attachment 67456
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67456&action=edit
Bug 14826; Unit Tests

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #95 from Tomás Cohen Arazi  ---
Created attachment 67455
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67455&action=edit
Bug 14826: Resurrect account offsets table

The account offsets table should be used to track increments and
decrements of fines via payments and credits, as well as fine accruals.
It should be able to match fees to payments and visa versa, so we can
know which fee was paid by a given payment, and which payments applied
to a given fee.

Test Plan:
1) Apply this patch
2) Run updatedatabase
3) Note the table accountoffsets has been renamed to account_offsets
4) Ensure fine generation creates offsets
5) Ensure creating a manual invoice creates an offset
6) Ensure a lost item charge creates an offset
7) Ensure Reverse Payment creates an offset
8) Ensure a payment creates an offset
9) Ensure a payment for multiple fees creates an offset for each
10) Ensure writeoffs create offsets

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #94 from Tomás Cohen Arazi  ---
Created attachment 67454
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67454&action=edit
Bug 14826: Add Koha::Account::Offset(s) modules

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #66387|0   |1
is obsolete||
  Attachment #66388|0   |1
is obsolete||
  Attachment #66389|0   |1
is obsolete||
  Attachment #66390|0   |1
is obsolete||
  Attachment #66391|0   |1
is obsolete||
  Attachment #66392|0   |1
is obsolete||
  Attachment #66393|0   |1
is obsolete||
  Attachment #66394|0   |1
is obsolete||
  Attachment #66395|0   |1
is obsolete||
  Attachment #66396|0   |1
is obsolete||
  Attachment #66397|0   |1
is obsolete||
  Attachment #66716|0   |1
is obsolete||

--- Comment #93 from Tomás Cohen Arazi  ---
Created attachment 67453
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67453&action=edit
Bug 14826: Update database

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-09-01 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #92 from Kyle M Hall  ---
Created attachment 66716
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66716&action=edit
Bug 14826 - Add account offset type table

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-28 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Kyle M Hall  changed:

   What|Removed |Added

 Blocks||19191


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19191
[Bug 19191] Add ability to email receipts for account payments and write-offs
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

 Status|Signed Off  |Passed QA

--- Comment #91 from Tomás Cohen Arazi  ---
This patch set is actually pretty simple and is not changing any behaviour
(besides the side effect of recording the account offsets on each operation
that involves charges/payments).

It passes the usual QA tests, and opens the door for really cool and missing
features in Koha (see the blocked bugs and it becomes obvious).

My only observation would be that the 'type' column values should somehow be
better controlled (possibly an ENUM, or better a FK to a separate table). This
would set a clearer path for eventual added flexibility and avoiding
migration/upgrades nightmares.

Congrats Kyle!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #65652|0   |1
is obsolete||

--- Comment #90 from Tomás Cohen Arazi  ---
Created attachment 66397
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66397&action=edit
Bug 14826 [QA Followup] - Only use plural modules in other modules

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #65650|0   |1
is obsolete||

--- Comment #88 from Tomás Cohen Arazi  ---
Created attachment 66395
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66395&action=edit
Bug 14826 - Update existing schema files

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #65651|0   |1
is obsolete||

--- Comment #89 from Tomás Cohen Arazi  ---
Created attachment 66396
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66396&action=edit
Bug 14826 [QA Followup] - Fix minor issues with unit tests

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #65648|0   |1
is obsolete||

--- Comment #86 from Tomás Cohen Arazi  ---
Created attachment 66393
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66393&action=edit
Bug 14826 - Add offset for rental fees

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #65649|0   |1
is obsolete||

--- Comment #87 from Tomás Cohen Arazi  ---
Created attachment 66394
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66394&action=edit
Bug 14826 - Add unit tests for _FixAccountForLostAndReturned and
_FixOverduesOnReturn

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #65647|0   |1
is obsolete||

--- Comment #85 from Tomás Cohen Arazi  ---
Created attachment 66392
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66392&action=edit
Bug 14826 - Add new offset table schema file

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #65646|0   |1
is obsolete||

--- Comment #84 from Tomás Cohen Arazi  ---
Created attachment 66391
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66391&action=edit
Bug 14826 - Remove old offset table schema file

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #65645|0   |1
is obsolete||

--- Comment #83 from Tomás Cohen Arazi  ---
Created attachment 66390
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66390&action=edit
Bug 14826 - Unit Tests

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #65644|0   |1
is obsolete||

--- Comment #82 from Tomás Cohen Arazi  ---
Created attachment 66389
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66389&action=edit
Bug 14826 - Resurrect account offsets table

The account offsets table should be used to track increments and
decrements of fines via payments and credits, as well as fine accruals.
It should be able to match fees to payments and visa versa, so we can
know which fee was paid by a given payment, and which payments applied
to a given fee.

Test Plan:
1) Apply this patch
2) Run updatedatabase
3) Note the table accountoffsets has been renamed to account_offsets
4) Ensure fine generation creates offsets
5) Ensure creating a manual invoice creates an offset
6) Ensure a lost item charge creates an offset
7) Ensure Reverse Payment creates an offset
8) Ensure a payment creates an offset
9) Ensure a payment for multiple fees creates an offset for each
10) Ensure writeoffs create offsets

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #65643|0   |1
is obsolete||

--- Comment #81 from Tomás Cohen Arazi  ---
Created attachment 66388
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66388&action=edit
Bug 14826 - Add Koha modules for offsets

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #65642|0   |1
is obsolete||

--- Comment #80 from Tomás Cohen Arazi  ---
Created attachment 66387
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66387&action=edit
Bug 14826 - Update database

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-11 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Tomás Cohen Arazi  changed:

   What|Removed |Added

Version|unspecified |master

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Aleisha Amohia  changed:

   What|Removed |Added

 CC|alei...@catalyst.net.nz |

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #78 from Kyle M Hall  ---
Created attachment 65651
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65651&action=edit
Bug 14826 [QA Followup] - Fix minor issues with unit tests

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #79 from Kyle M Hall  ---
Created attachment 65652
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65652&action=edit
Bug 14826 [QA Followup] - Only use plural modules in other modules

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #76 from Kyle M Hall  ---
Created attachment 65649
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65649&action=edit
Bug 14826 - Add unit tests for _FixAccountForLostAndReturned and
_FixOverduesOnReturn

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #77 from Kyle M Hall  ---
Created attachment 65650
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65650&action=edit
Bug 14826 - Update existing schema files

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #75 from Kyle M Hall  ---
Created attachment 65648
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65648&action=edit
Bug 14826 - Add offset for rental fees

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #74 from Kyle M Hall  ---
Created attachment 65647
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65647&action=edit
Bug 14826 - Add new offset table schema file

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #72 from Kyle M Hall  ---
Created attachment 65645
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65645&action=edit
Bug 14826 - Unit Tests

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #73 from Kyle M Hall  ---
Created attachment 65646
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65646&action=edit
Bug 14826 - Remove old offset table schema file

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #71 from Kyle M Hall  ---
Created attachment 65644
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65644&action=edit
Bug 14826 - Resurrect account offsets table

The account offsets table should be used to track increments and
decrements of fines via payments and credits, as well as fine accruals.
It should be able to match fees to payments and visa versa, so we can
know which fee was paid by a given payment, and which payments applied
to a given fee.

Test Plan:
1) Apply this patch
2) Run updatedatabase
3) Note the table accountoffsets has been renamed to account_offsets
4) Ensure fine generation creates offsets
5) Ensure creating a manual invoice creates an offset
6) Ensure a lost item charge creates an offset
7) Ensure Reverse Payment creates an offset
8) Ensure a payment creates an offset
9) Ensure a payment for multiple fees creates an offset for each
10) Ensure writeoffs create offsets

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #70 from Kyle M Hall  ---
Created attachment 65643
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65643&action=edit
Bug 14826 - Add Koha modules for offsets

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Kyle M Hall  changed:

   What|Removed |Added

  Attachment #65459|0   |1
is obsolete||
  Attachment #65460|0   |1
is obsolete||
  Attachment #65461|0   |1
is obsolete||
  Attachment #65462|0   |1
is obsolete||
  Attachment #65463|0   |1
is obsolete||
  Attachment #65464|0   |1
is obsolete||
  Attachment #65465|0   |1
is obsolete||
  Attachment #65466|0   |1
is obsolete||
  Attachment #65467|0   |1
is obsolete||

--- Comment #69 from Kyle M Hall  ---
Created attachment 65642
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65642&action=edit
Bug 14826 - Update database

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-08 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Josef Moravec  changed:

   What|Removed |Added

 Blocks||19036


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19036
[Bug 19036] Numer payment receipts / payment slips
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-03 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #64 from Kyle M Hall  ---
Created attachment 65463
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65463&action=edit
Bug 14826 - Remove old offset table schema file

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-03 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #67 from Kyle M Hall  ---
Created attachment 65466
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65466&action=edit
Bug 14826 - Add unit tests for _FixAccountForLostAndReturned and
_FixOverduesOnReturn

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-03 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #68 from Kyle M Hall  ---
Created attachment 65467
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65467&action=edit
Bug 14826 - Update existing schema files

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-03 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #61 from Kyle M Hall  ---
Created attachment 65460
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65460&action=edit
Bug 14826 - Add Koha modules for offsets

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-03 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #66 from Kyle M Hall  ---
Created attachment 65465
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65465&action=edit
Bug 14826 - Add offset for rental fees

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-03 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #65 from Kyle M Hall  ---
Created attachment 65464
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65464&action=edit
Bug 14826 - Add new offset table schema file

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-03 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #62 from Kyle M Hall  ---
Created attachment 65461
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65461&action=edit
Bug 14826 - Resurrect account offsets table

The account offsets table should be used to track increments and
decrements of fines via payments and credits, as well as fine accruals.
It should be able to match fees to payments and visa versa, so we can
know which fee was paid by a given payment, and which payments applied
to a given fee.

Test Plan:
1) Apply this patch
2) Run updatedatabase
3) Note the table accountoffsets has been renamed to account_offsets
4) Ensure fine generation creates offsets
5) Ensure creating a manual invoice creates an offset
6) Ensure a lost item charge creates an offset
7) Ensure Reverse Payment creates an offset
8) Ensure a payment creates an offset
9) Ensure a payment for multiple fees creates an offset for each
10) Ensure writeoffs create offsets

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-03 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

--- Comment #63 from Kyle M Hall  ---
Created attachment 65462
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65462&action=edit
Bug 14826 - Unit Tests

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-08-03 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Kyle M Hall  changed:

   What|Removed |Added

  Attachment #64217|0   |1
is obsolete||
  Attachment #64220|0   |1
is obsolete||
  Attachment #64222|0   |1
is obsolete||
  Attachment #64223|0   |1
is obsolete||
  Attachment #64224|0   |1
is obsolete||
  Attachment #64225|0   |1
is obsolete||
  Attachment #64226|0   |1
is obsolete||
  Attachment #64227|0   |1
is obsolete||
  Attachment #64228|0   |1
is obsolete||

--- Comment #60 from Kyle M Hall  ---
Created attachment 65459
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=65459&action=edit
Bug 14826 - Update database

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 14826] Resurrect account offsets table

2017-07-07 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14826

Martha Fuerst  changed:

   What|Removed |Added

 CC||mfue...@hmcpl.org

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


  1   2   >