[Koha-bugs] [Bug 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Galen Charlton changed: What|Removed |Added Status|Passed QA |Pushed to Master CC||gmcha...@gmail.com Patch complexity|--- |Medium patch --- Comment #18 from Galen Charlton --- Pushed to master. Thanks, 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Jonathan Druart changed: What|Removed |Added Attachment #18155|0 |1 is obsolete|| Attachment #18273|0 |1 is obsolete|| --- Comment #16 from Jonathan Druart --- Created attachment 18307 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18307&action=edit Bug 10080 - Change system pref IndependantBranches to IndependentBranches Test Plan: 1) Enable IndependantBranches 2) Apply this patch 3) Run updatedatabase.pl 4) Verify that the system preference still functions correctly Signed-off-by: Marcel de Rooy Signed-off-by: Jonathan Druart -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 --- Comment #17 from Jonathan Druart --- Created attachment 18308 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18308&action=edit Bug 10080 - Change system pref IndependantBranches to IndependentBranches - Followup Signed-off-by: Jonathan Druart -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Jonathan Druart changed: What|Removed |Added Status|Signed Off |Passed QA QA Contact|m.de.r...@rijksmuseum.nl|jonathan.dru...@biblibre.co ||m --- Comment #15 from Jonathan Druart --- QA comment: Looks good to me now. The only occurrences are git grep -l IndependantBranches | grep -v translator installer/data/mysql/updatedatabase.pl misc/release_notes/release_notes_30004.txt misc/release_notes/release_notes_3_10_0.txt misc/release_notes/release_notes_3_12_0.txt We have to keep this patch in mind for future patch! I will provide a patch for the qa tools. Marked as 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 --- Comment #14 from Kyle M Hall --- Created attachment 18273 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18273&action=edit Bug 10080 - Change system pref IndependantBranches to IndependentBranches - Followup -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Kyle M Hall changed: What|Removed |Added Status|Failed QA |Signed Off -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #13 from Katrin Fischer --- Sorry Kyle, I finally took a step forward in direction of understanding how to grep and found another occurence: /reserve/request.pl. Please provide a follow up. -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 --- Comment #12 from Kyle M Hall --- (In reply to comment #10) > Hi Kyle, > > your patch contains the po files, but changes to those files should be > handled via Pootle. Can you please resubmit your patch without them? > > I am also not sure if old release notes should be included, as they are > 'history'. Good call, fixed! -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Kyle M Hall changed: What|Removed |Added Attachment #18044|0 |1 is obsolete|| --- Comment #11 from Kyle M Hall --- Created attachment 18155 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18155&action=edit Bug 10080 - Change system pref IndependantBranches to IndependentBranches Test Plan: 1) Enable IndependantBranches 2) Apply this patch 3) Run updatedatabase.pl 4) Verify that the system preference still functions correctly 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Kyle M Hall changed: What|Removed |Added Status|Failed QA |Signed Off -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #10 from Katrin Fischer --- Hi Kyle, your patch contains the po files, but changes to those files should be handled via Pootle. Can you please resubmit your patch without them? I am also not sure if old release notes should be included, as they are 'history'. -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Katrin Fischer changed: What|Removed |Added CC||katrin.fisc...@bsz-bw.de Version|master |3.14 --- Comment #9 from Katrin Fischer --- I second Chris' vote to hold this patch for 3.14 to not risk missing something, marking bug accordingly. -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 --- Comment #8 from Kyle M Hall --- Updated patch, removed modification for the previous database update. Thanks for catching that! -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Kyle M Hall changed: What|Removed |Added Blocks||10200 -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Kyle M Hall changed: What|Removed |Added Attachment #17588|0 |1 is obsolete|| --- Comment #7 from Kyle M Hall --- Created attachment 18044 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18044&action=edit Bug 10080 - Change system pref IndependantBranches to IndependentBranches Test Plan: 1) Enable IndependantBranches 2) Apply this patch 3) Run updatedatabase.pl 4) Verify that the system preference still functions correctly 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Kyle M Hall changed: What|Removed |Added Status|Failed QA |Signed Off -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Chris Cormack changed: What|Removed |Added CC||ch...@bigballofwax.co.nz --- Comment #6 from Chris Cormack --- This seems like a very dangerous change to make with minimum benefits this close to a major release. I vote it is held over and not rebased until post release. It gains users nothing, with the risk of breaking lots. -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 --- Comment #5 from M. de Rooy --- (In reply to comment #4) > I disagree with a change in updatedatabase.pl: > > There is a modification of an old version ($DBversion = "3.09.00.047"). When > this test will be executed, the syspref will not be renamed and will always > return 0. > > Switch back to Passed QA if I missed something. Good catch. Thanks. I agree with you. Kyle: could you send a followup eliminating the change of the older db revision. Only leave the new one under 3.11.00.XXX in updatedatabase. After that please switch back to Signed off. I will test 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Failed QA CC||jonathan.dru...@biblibre.co ||m --- Comment #4 from Jonathan Druart --- I disagree with a change in updatedatabase.pl: There is a modification of an old version ($DBversion = "3.09.00.047"). When this test will be executed, the syspref will not be renamed and will always return 0. Switch back to Passed QA if I missed something. -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 M. de Rooy changed: What|Removed |Added QA Contact||m.de.r...@rijksmuseum.nl -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 M. de Rooy changed: What|Removed |Added Status|Signed Off |Passed QA CC||m.de.r...@rijksmuseum.nl --- Comment #3 from M. de Rooy --- QA Comment: Tested the db update. Grepped on the old term (only in updatedatabase now). Tested with superlibrarian and normal user in both modes. Nothing unusual. Looks good to me. Jared: Due to number of affected files, this patch would need a lot of rebasing if it stays too long in the queue.. 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 M. de Rooy changed: What|Removed |Added Status|Needs Signoff |Signed Off -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 M. de Rooy changed: What|Removed |Added Attachment #17556|0 |1 is obsolete|| --- Comment #2 from M. de Rooy --- Created attachment 17588 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=17588&action=edit Bug 10080 - Change system pref IndependantBranches to IndependentBranches Test Plan: 1) Enable IndependantBranches 2) Apply this patch 3) Run updatedatabase.pl 4) Verify that the system preference still functions correctly 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 --- Comment #1 from Kyle M Hall --- Created attachment 17556 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=17556&action=edit Bug 10080 - Change system pref IndependantBranches to IndependentBranches Test Plan: 1) Enable IndependantBranches 2) Apply this patch 3) Run updatedatabase.pl 4) Verify that the system preference still functions correctly -- 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Kyle M Hall 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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Kyle M Hall changed: What|Removed |Added Assignee|gmcha...@gmail.com |kyle.m.h...@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 10080] Change system pref IndependantBranches to IndependentBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10080 Kyle M Hall changed: What|Removed |Added Severity|enhancement |trivial -- 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/
