[Koha-bugs] [Bug 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Sophie MEYNIEUX changed: What|Removed |Added Attachment #22542|0 |1 is obsolete|| --- Comment #5 from Sophie MEYNIEUX --- Created attachment 25212 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25212&action=edit Bug 7372: Move road types from the roadtype table to the ROADTYPE AV Currently road types are stored in a specific table in DB. Moreover, an admin page is present in order to manage them. This patch proposes to remove this table and this page in favour of a new authorised value category 'ROADTYPE'. This patch: - adds a new AV category 'ROADTYPE' (created from the roadtype table content). - remove the roadtype table. - remove the .pl and .tt file admin/roadtype - remove the 2 routines C4::Members::GetRoadTypes and C4::Members::GetRoadTypeDetails Test plan: 1/ Execute the updatedatabase entry and verify existing roadtypes are now stored in the AV 'ROADTYPE'. 2/ Verify you can add/update a streettype for patrons. 3/ Verify on following pages the streettype is displayed in patron information (top left): circ/circulation.pl members/memberentry.pl members/moremember.pl members/routing-lists.pl Signed-off-by: Sophie Meynieux -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Sophie MEYNIEUX changed: What|Removed |Added Status|Needs Signoff |Signed Off CC||sophie.meyni...@biblibre.co ||m -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 --- Comment #6 from Sophie MEYNIEUX --- Works as described in test plan -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Sophie MEYNIEUX changed: What|Removed |Added Blocks||11760 -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Kyle M Hall changed: What|Removed |Added Attachment #25212|0 |1 is obsolete|| --- Comment #7 from Kyle M Hall --- Created attachment 25540 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25540&action=edit Bug 7372: Move road types from the roadtype table to the ROADTYPE AV Currently road types are stored in a specific table in DB. Moreover, an admin page is present in order to manage them. This patch proposes to remove this table and this page in favour of a new authorised value category 'ROADTYPE'. This patch: - adds a new AV category 'ROADTYPE' (created from the roadtype table content). - remove the roadtype table. - remove the .pl and .tt file admin/roadtype - remove the 2 routines C4::Members::GetRoadTypes and C4::Members::GetRoadTypeDetails Test plan: 1/ Execute the updatedatabase entry and verify existing roadtypes are now stored in the AV 'ROADTYPE'. 2/ Verify you can add/update a streettype for patrons. 3/ Verify on following pages the streettype is displayed in patron information (top left): circ/circulation.pl members/memberentry.pl members/moremember.pl members/routing-lists.pl Signed-off-by: Sophie Meynieux Signed-off-by: Kyle M Hall Passes koha-qa.pl, t and xt. Works as advertised. -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Kyle M Hall changed: What|Removed |Added Status|Signed Off |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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Galen Charlton changed: What|Removed |Added Status|Passed QA |Pushed to Master CC||gmcha...@gmail.com --- Comment #8 from Galen Charlton --- Pushed to master. 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Jonathan Druart changed: What|Removed |Added CC||jonathan.dru...@biblibre.co ||m -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Jonathan Druart changed: What|Removed |Added Status|NEW |Needs Signoff -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 --- Comment #2 from Jonathan Druart --- Created attachment 22310 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22310&action=edit Bug 7372: Move road types from the roadtype table to the ROADTYPE AV Currently road types are stored in a specific table in DB. Moreover, an admin page is present in order to manage them. This patch proposes to remove this table and this page in favour of a new authorised value category 'ROADTYPE'. This patch: - adds a new AV category 'ROADTYPE' (created from the roadtype table content). - remove the roadtype table. - remove the .pl and .tt file admin/roadtype - remove the 2 routines C4::Members::GetRoadTypes and C4::Members::GetRoadTypeDetails Test plan: 1/ Execute the updatedatabase entry and verify existing roadtypes are now stored in the AV 'ROADTYPE'. 2/ Verify you can add/update a streettype for patrons. 3/ Verify on following pages the streettype is displayed in patron information (top left): circ/circulation.pl members/memberentry.pl members/moremember.pl members/routing-lists.pl -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Jonathan Druart changed: What|Removed |Added Version|3.8 |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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Jonathan Druart changed: What|Removed |Added Assignee|koha.sek...@gmail.com |jonathan.dru...@biblibre.co ||m QA Contact|jonathan.dru...@biblibre.co | |m | -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 I'm just a bot changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply CC||git...@bugs.koha-community. ||org When did the bot||2013-10-30 last check this|| --- Comment #3 from I'm just a bot --- Applying: Bug 7372: Move road types from the roadtype table to the ROADTYPE AV Using index info to reconstruct a base tree... Minstaller/data/mysql/updatedatabase.pl Falling back to patching base and 3-way merge... Removing koha-tmpl/intranet-tmpl/prog/en/modules/admin/roadtype.tt Auto-merging installer/data/mysql/updatedatabase.pl CONFLICT (content): Merge conflict in installer/data/mysql/updatedatabase.pl Removing admin/roadtype.pl Patch failed at 0001 Bug 7372: Move road types from the roadtype table to the ROADTYPE AV The copy of the patch that failed is found in: /home/christopher/git/koha/.git/rebase-apply/patch When you have resolved this problem, run "git am --continue". If you prefer to skip this patch, run "git am --skip" instead. To restore the original branch and stop patching, run "git am --abort". -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Chris Cormack changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Chris Cormack changed: What|Removed |Added Attachment #22310|0 |1 is obsolete|| --- Comment #4 from Chris Cormack --- Created attachment 22542 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22542&action=edit Bug 7372: Move road types from the roadtype table to the ROADTYPE AV Currently road types are stored in a specific table in DB. Moreover, an admin page is present in order to manage them. This patch proposes to remove this table and this page in favour of a new authorised value category 'ROADTYPE'. This patch: - adds a new AV category 'ROADTYPE' (created from the roadtype table content). - remove the roadtype table. - remove the .pl and .tt file admin/roadtype - remove the 2 routines C4::Members::GetRoadTypes and C4::Members::GetRoadTypeDetails Test plan: 1/ Execute the updatedatabase entry and verify existing roadtypes are now stored in the AV 'ROADTYPE'. 2/ Verify you can add/update a streettype for patrons. 3/ Verify on following pages the streettype is displayed in patron information (top left): circ/circulation.pl members/memberentry.pl members/moremember.pl members/routing-lists.pl -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Nicole C. Engard changed: What|Removed |Added CC||neng...@gmail.com -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Fridolin SOMERS changed: What|Removed |Added CC||fridolyn.som...@biblibre.co ||m Patch complexity|--- |Small patch -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Fridolin SOMERS changed: What|Removed |Added Status|Pushed to Master|Pushed to Stable --- Comment #9 from Fridolin SOMERS --- Pushed to 3.14.x, will be in 3.14.08 -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 --- Comment #10 from Jonathan Druart --- Created attachment 31526 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=31526&action=edit Bug 7372: (follow-up) does execute this DB entry twice Since this enh has been backported on 3.14, this patch should be applied on 3.16 and master branch in order to update the updatedb entry. It should not be executed twice otherwise the process is stuck on this entry. http://irc.koha-community.org/koha/2014-09-11#i_1567078 -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Jonathan Druart changed: What|Removed |Added CC||tomasco...@gmail.com --- Comment #11 from Jonathan Druart --- ping 3.16 and master RM. -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 --- Comment #12 from Tomás Cohen Arazi --- Followup pushed to master. Next time please fill a new bug report! -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Tomás Cohen Arazi changed: What|Removed |Added Status|Pushed to Stable|Pushed to Master Version|master |3.16 -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Katrin Fischer changed: What|Removed |Added Keywords||rel_3_16_4_candidate CC||katrin.fisc...@bsz-bw.de -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 --- Comment #13 from Jonathan Druart --- Created attachment 32083 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=32083&action=edit Bug 7372: (follow-up) remove debug message -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 --- Comment #14 from Jonathan Druart --- Tomas, Paul have found a debug message (in French...) in the last patch. Could you push it please? -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 --- Comment #15 from Tomás Cohen Arazi --- Followup pushed to master, 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 --- Comment #1 from Ian Walls --- Data layer: Step 1: create new authorised value ROADTYPE, both in default SQL and databaseupdate.pl Step 2: migrate existing roadtype table into new authorised value category using databaseupdate.pl Code layer: Step 3: replace usage of GetRoadTypes and GetRoadTypeDetails with GetAuthorisedValues(ROADTYPE) and GetKohaAuthorisedValueLib() respectively Step 4: remove administrative links to admin/roadtype.pl Cleanup: Step 5: remove admin/roadtype.pl page and corresponding template Step 6: remove GetRoadTypes and GetRoadTypeDetails subroutines from C4/Members.pm Step 7: remove roadtype table from DB and kohastructure.sql Documentation: Step 8: update documentation to reflect new mechanism for handling Road Types -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Mason James changed: What|Removed |Added CC||m...@kohaaloha.com --- Comment #16 from Mason James --- (In reply to Jonathan Druart from comment #11) > ping 3.16 and master RM. Followups pushed to 3.16.x, will be in 3.16.6 -- 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 7372] RoadTypes should be folded into Authorised Values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7372 Mason James changed: What|Removed |Added Blocks||13443 Referenced Bugs: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13443 [Bug 13443] DBD::mysql::db do failed: Table 'koha_library.roadtype' doesn't exist -- 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/