[Koha-bugs] [Bug 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Roch D'Amour 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Roch D'Amour changed: What|Removed |Added Attachment #69985|0 |1 is obsolete|| --- Comment #7 from Roch D'Amour --- Created attachment 70780 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70780&action=edit Bug 19230: Preventing warn when deleting course To test: 1) Ensure UseCourseReserves is enabled 2) Go to Course Reserves, create a course 3) When you are taken to the course page, you should notice the 'Argument "" isn't numeric in numeric gt' warn 4) Delete course 5) Notice warn, and that no confirm message popped up 6) Apply patch 7) Create a course again. Confirm the 'Argument "" isn't numeric in numeric gt' warn did not show 8) Delete the course 9) Notice no warn and the confirm message pops up Sponsored-by: Catalyst IT Signed-off-by: Roch D'Amour -- 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Marcel de Rooy changed: What|Removed |Added Attachment #70780|0 |1 is obsolete|| --- Comment #8 from Marcel de Rooy --- Created attachment 70933 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70933&action=edit Bug 19230: Preventing warn when deleting course To test: 1) Ensure UseCourseReserves is enabled 2) Go to Course Reserves, create a course 3) When you are taken to the course page, you should notice the 'Argument "" isn't numeric in numeric gt' warn 4) Delete course 5) Notice warn, and that no confirm message popped up 6) Apply patch 7) Create a course again. Confirm the 'Argument "" isn't numeric in numeric gt' warn did not show 8) Delete the course 9) Notice no warn and the confirm message pops up Sponsored-by: Catalyst IT Signed-off-by: Roch D'Amour 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Marcel de Rooy 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Marcel de Rooy changed: What|Removed |Added CC||m.de.r...@rijksmuseum.nl QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl |y.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/
[Koha-bugs] [Bug 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 --- Comment #9 from Jonathan Druart --- Created attachment 71067 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=71067&action=edit Bug 19230: (follow-up) Preventing warn when deleting course Simplify 'unless' statement -- 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #10 from Jonathan Druart --- Pushed to master for 18.05, 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Nick Clemens changed: What|Removed |Added CC||n...@bywatersolutions.com Status|Pushed to Master|Pushed to Stable --- Comment #11 from Nick Clemens --- Awesome work all! This patch has been pushed to 17.11.x and will be in 17.11.03 -- 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 --- Comment #1 from Aleisha Amohia --- Created attachment 66678 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66678&action=edit Bug 19230: Preventing warn when deleting course To test: 1) Ensure UseCourseReserves is enabled 2) Go to Course Reserves, create a course 3) Delete course 4) Notice warn 5) Apply patch 6) Create a course again and delete it 7) Notice no warn Sponsored-by: Catalyst IT -- You are receiving this mail because: You are the assignee for the bug. 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Aleisha Amohia changed: What|Removed |Added Patch complexity|--- |Small patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Aleisha Amohia changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |alei...@catalyst.net.nz |ity.org | Change sponsored?|--- |Sponsored -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Katrin Fischer changed: What|Removed |Added Attachment #66678|0 |1 is obsolete|| --- Comment #2 from Katrin Fischer --- Created attachment 67797 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67797&action=edit Bug 19230: Preventing warn when deleting course To test: 1) Ensure UseCourseReserves is enabled 2) Go to Course Reserves, create a course 3) Delete course 4) Notice warn 5) Apply patch 6) Create a course again and delete it 7) Notice no warn Sponsored-by: Catalyst IT Signed-off-by: Katrin Fischer -- 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Katrin Fischer 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Tomás Cohen Arazi changed: What|Removed |Added CC||tomasco...@gmail.com Severity|enhancement |minor 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Jonathan Druart changed: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #3 from Jonathan Druart --- I think it would be better to assign a $course_id variable and reuse it. Something like: modified: course_reserves/mod_course.pl @ mod_course.pl:41 @ my ( $template, $loggedinuser, $cookie ) = get_template_and_user( ); my $action = $cgi->param('action') || ''; +my $course_id = $cgi->param('course_id'); if ( $action eq 'del' ) { -DelCourse( $cgi->param('course_id') ); +DelCourse( $course_id ); print $cgi->redirect("/cgi-bin/koha/course_reserves/course-reserves.pl"); } else { my %params; -$params{'course_id'} = $cgi->param('course_id') - if ( $cgi->param('course_id') ); +$params{'course_id'} = $course_id; $params{'department'} = $cgi->param('department'); $params{'course_number'} = $cgi->param('course_number'); $params{'section'}= $cgi->param('section'); @ mod_course.pl:60 @ if ( $action eq 'del' ) { $params{'students_count'} = $cgi->param('students_count'); $params{'enabled'}= ( $cgi->param('enabled') eq 'on' ) ? 'yes' : 'no'; -my $course_id = ModCourse(%params); +my $new_course_id = ModCourse(%params); my @instructors = $cgi->multi_param('instructors'); ModCourseInstructors( mode=> 'replace', cardnumbers => \@instructors, -course_id => $course_id +course_id => $new_course_id ); -print $cgi->redirect("/cgi-bin/koha/course_reserves/course-details.pl?course_id=$course_id"); +print $cgi->redirect("/cgi-bin/koha/course_reserves/course-details.pl?course_id=$new_course_id"); } Would it make more sense? -- 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Katrin Fischer changed: What|Removed |Added Attachment #67797|0 |1 is obsolete|| --- Comment #4 from Katrin Fischer --- Created attachment 69946 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=69946&action=edit Bug 19230: Preventing warn when deleting course To test: 1) Ensure UseCourseReserves is enabled 2) Go to Course Reserves, create a course 3) Delete course 4) Notice warn 5) Apply patch 6) Create a course again and delete it 7) Notice no warn Sponsored-by: Catalyst IT Signed-off-by: Katrin Fischer -- 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Katrin Fischer 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Jonathan Druart changed: What|Removed |Added Status|Passed 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 --- Comment #5 from Aleisha Amohia --- (In reply to Jonathan Druart from comment #3) > Would it make more sense? hi Jonathan, Yes I believe your fix makes more sense so I have implemented in the next patch. When I was testing, I also found a new warn when looking at a course with NO course reserves in it (course_reserves/course-details.pl) Argument "" isn't numeric in numeric gt (>) at /home/vagrant/kohaclone/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt line 219. Also, when deleting a course with no course reserves, the confirm message is not triggered. I've added a fix for this also in my next 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Aleisha Amohia changed: What|Removed |Added Attachment #69946|0 |1 is obsolete|| --- Comment #6 from Aleisha Amohia --- Created attachment 69985 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=69985&action=edit Bug 19230: Preventing warn when deleting course To test: 1) Ensure UseCourseReserves is enabled 2) Go to Course Reserves, create a course 3) When you are taken to the course page, you should notice the 'Argument "" isn't numeric in numeric gt' warn 4) Delete course 5) Notice warn, and that no confirm message popped up 6) Apply patch 7) Create a course again. Confirm the 'Argument "" isn't numeric in numeric gt' warn did not show 8) Delete the course 9) Notice no warn and the confirm message pops up Sponsored-by: Catalyst 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 19230] Warn when deleting a course in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19230 Aleisha Amohia changed: What|Removed |Added Status|Signed Off |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/