This sub is even commented out for years.
Created attachment 81259 [details] [review] Bug 21680: Remove dead code - C4::Accounts::fixaccounts Test plan: git grep fixacccounts -- without patch there are 3 occurences -- with patch - no occurence of fixaccounts Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 81295 [details] [review] Bug 21680: Remove dead code - C4::Accounts::fixaccounts Test plan: git grep fixacccounts -- without patch there are 3 occurences -- with patch - no occurence of fixaccounts Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Well found.. you are entirely right, I can't find a single case across the whole codebase where this is used now. Going straight for PQA.
Awesome work all! Pushed to master for 18.11
Pushed to 18.05.x for 18.05.06.. always good to trim the fat.
I prefer not to impact 17.11.x