[Koha-bugs] [Bug 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 Fridolin SOMERS changed: What|Removed |Added Status|Pushed to Master|Pushed to Stable CC||fridolyn.som...@biblibre.co ||m --- Comment #41 from Fridolin SOMERS --- Those patches has been pushed to 3.14.x, will be in 3.14.1. (integration into master added a commit : "1be884f Bug 7791: (follow-up) adjust wording of basket deletion confirmation dialog") -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 Galen Charlton changed: What|Removed |Added Status|Passed QA |Pushed to Master CC||gmcha...@gmail.com --- Comment #40 from Galen Charlton --- Pushed to master. Thanks, Mathieu! -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #39 from mathieu saby --- (In reply to M. de Rooy from comment #38) > QA Comment: > Works as advertised. Nice to have. > No complaints from qa tools. Code looks good to me. > Includes string changes, so candidate for 3.16. > I am passing QA on these patches, but have some hesitations as to design and > navigation, as described in the following points: > > 1) The design of the page showing that the basket has been deleted, but also > listing a warning c/should look better (layman's perspective :) > 2) This same error page gives me the link Go back to booksellers page (note > the problems reported earlier about its performance). Since I came from a > basket of a specific bookseller, I would rather go back to the basket list > of THAT bookseller. You opened up a new report for that, I understand. I > think that this set of patches would be even more attractive with such a > link. > 3) A successful deletion actually does the same and sends me back to all > booksellers. Please give me my particular bookseller instead :) > > Thanks. > > PS I downgraded the bug severity. I can understand that you feel this is an > important issue, but critical is really another category. Thank you For the points 1-3 it could be improved by other patches ;-) Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 M. de Rooy changed: What|Removed |Added Status|Signed Off |Passed QA Patch complexity|Small patch |Medium patch Severity|critical|normal --- Comment #38 from M. de Rooy --- QA Comment: Works as advertised. Nice to have. No complaints from qa tools. Code looks good to me. Includes string changes, so candidate for 3.16. I am passing QA on these patches, but have some hesitations as to design and navigation, as described in the following points: 1) The design of the page showing that the basket has been deleted, but also listing a warning c/should look better (layman's perspective :) 2) This same error page gives me the link Go back to booksellers page (note the problems reported earlier about its performance). Since I came from a basket of a specific bookseller, I would rather go back to the basket list of THAT bookseller. You opened up a new report for that, I understand. I think that this set of patches would be even more attractive with such a link. 3) A successful deletion actually does the same and sends me back to all booksellers. Please give me my particular bookseller instead :) Thanks. PS I downgraded the bug severity. I can understand that you feel this is an important issue, but critical is really another category. -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791
M. de Rooy changed:
What|Removed |Added
Attachment #22093|0 |1
is obsolete||
--- Comment #37 from M. de Rooy ---
Created attachment 22766
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22766&action=edit
Bug 7791 Followup: add warning after deletion if some records were not deleted
This followup add some warnings after deletion if some items were not deleted.
4 types of messages are possible :
- x item(s) attached.
- x subscription(s) attached.
- x order(s) attached.
- Unknown error.
To test :
test a
1. create a basket with
- an order using a record A which has already an item, which is used in a
subscription, and in other order (in an other basket)
- an order using a record B used nowhere elese
2. click on "Delete basket"
3. choose button "Delete basket, orders and records"
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item, subscription and order.
5. check the link around the title of record B takes you to the record
6. check the link under the warning box ("Click here to go back to booksellers
page") takes you to booksellers page
5. check record B is deleted
test b
1. suppress the subscription linked with record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item and order
test c
1. suppress the item attached under record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its orderBug 7791 Followup: add warning after deletion if
some
records were not deleted
Content-Type: text/plain; charset="utf-8"
This followup add some warnings after deletion if some items were not deleted.
4 types of messages are possible :
- x item(s) attached.
- x subscription(s) attached.
- x order(s) attached.
- Unknown error.
To test :
test a
1. create a basket with
- an order using a record A which has already an item, which is used in a
subscription, and in other order (in an other basket)
- an order using a record B used nowhere elese
2. click on "Delete basket"
3. choose button "Delete basket, orders and records"
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item, subscription and order.
5. check the link around the title of record B takes you to the record
6. check the link under the warning box ("Click here to go back to booksellers
page") takes you to booksellers page
5. check record B is deleted
test b
1. suppress the subscription linked with record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item and order
test c
1. suppress the item attached under record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its order
Signed-off-by: Chris Cormack
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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 M. de Rooy changed: What|Removed |Added Attachment #22092|0 |1 is obsolete|| --- Comment #36 from M. de Rooy --- Created attachment 22765 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22765&action=edit Bug 7791 Followup: delete warns, add missing curly braket and make a simpler message if basket is void This patch - delete warns - add a missing } - add a condition in template of avoiding asking to suppress orders or records if the basket is void To test : 1. Make the same tests as defined in test plan of main patch. It should behave the same way 2. Try to delete a basket with no records inside. You will only have a "Delete basket" button, with less warnings Signed-off-by: Cedric Vita Signed-off-by: Chris Cormack 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 M. de Rooy changed: What|Removed |Added Attachment #22091|0 |1 is obsolete|| --- Comment #35 from M. de Rooy --- Created attachment 22764 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22764&action=edit Bug 7791 : Give the ability to delete records when deleting a basket Currently, when a basket is deleted, all the orders are deleted (there is a foreign key in aqorders table on basketno). This could be dangerous, and there is no warn. After the deletion, unused biblios are left in the catalogue. This patch - adds a more detailed message describing the consequences of deletion - give the choice of also deleting biblio records if possible To test : Test A : 1. create a basket with 4 orders: - an order from a new record A - an order from a record B which has already an item - an order from a record C used in a subscription - an order from a record D used in an other order 2. note the biblionumbers of the records used (or open them in other tabs in yout browser) 3. click on "Delete basket" 4. choose button "Delete basket and orders" 5. check the catalogue : records A,B,C,D must still be there Test B: 1. create a basket with 4 orders: - an order from a new record A - an order from a record B which has already an item - an order from a record C used in a subscription - an order from a record D used in an other order 2. note the biblionumbers of the records used (or open them in other tabs in yout browser) 3. click on "Delete basket" 4. choose button "Delete basket, orders and records" 5. check the catalogue : records B,C,D must still be there. Record A must be deleted Signed-off-by: Cedric Vita Signed-off-by: Chris Cormack 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 M. de Rooy changed: What|Removed |Added See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=10978 -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 M. de Rooy changed: What|Removed |Added See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=11215 -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791
Chris Cormack changed:
What|Removed |Added
Attachment #22046|0 |1
is obsolete||
--- Comment #34 from Chris Cormack ---
Created attachment 22093
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22093&action=edit
Bug 7791 Followup: add warning after deletion if some records were not deleted
This followup add some warnings after deletion if some items were not deleted.
4 types of messages are possible :
- x item(s) attached.
- x subscription(s) attached.
- x order(s) attached.
- Unknown error.
To test :
test a
1. create a basket with
- an order using a record A which has already an item, which is used in a
subscription, and in other order (in an other basket)
- an order using a record B used nowhere elese
2. click on "Delete basket"
3. choose button "Delete basket, orders and records"
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item, subscription and order.
5. check the link around the title of record B takes you to the record
6. check the link under the warning box ("Click here to go back to booksellers
page") takes you to booksellers page
5. check record B is deleted
test b
1. suppress the subscription linked with record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item and order
test c
1. suppress the item attached under record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its orderBug 7791 Followup: add warning after deletion if
some
records were not deleted
Content-Type: text/plain; charset="utf-8"
This followup add some warnings after deletion if some items were not deleted.
4 types of messages are possible :
- x item(s) attached.
- x subscription(s) attached.
- x order(s) attached.
- Unknown error.
To test :
test a
1. create a basket with
- an order using a record A which has already an item, which is used in a
subscription, and in other order (in an other basket)
- an order using a record B used nowhere elese
2. click on "Delete basket"
3. choose button "Delete basket, orders and records"
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item, subscription and order.
5. check the link around the title of record B takes you to the record
6. check the link under the warning box ("Click here to go back to booksellers
page") takes you to booksellers page
5. check record B is deleted
test b
1. suppress the subscription linked with record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item and order
test c
1. suppress the item attached under record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its order
Signed-off-by: Chris Cormack
--
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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 Chris Cormack 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 Chris Cormack changed: What|Removed |Added Attachment #21246|0 |1 is obsolete|| --- Comment #33 from Chris Cormack --- Created attachment 22092 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22092&action=edit Bug 7791 Followup: delete warns, add missing curly braket and make a simpler message if basket is void This patch - delete warns - add a missing } - add a condition in template of avoiding asking to suppress orders or records if the basket is void To test : 1. Make the same tests as defined in test plan of main patch. It should behave the same way 2. Try to delete a basket with no records inside. You will only have a "Delete basket" button, with less warnings Signed-off-by: Cedric Vita Signed-off-by: Chris Cormack -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 Chris Cormack changed: What|Removed |Added Attachment #21245|0 |1 is obsolete|| --- Comment #32 from Chris Cormack --- Created attachment 22091 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22091&action=edit Bug 7791 : Give the ability to delete records when deleting a basket Currently, when a basket is deleted, all the orders are deleted (there is a foreign key in aqorders table on basketno). This could be dangerous, and there is no warn. After the deletion, unused biblios are left in the catalogue. This patch - adds a more detailed message describing the consequences of deletion - give the choice of also deleting biblio records if possible To test : Test A : 1. create a basket with 4 orders: - an order from a new record A - an order from a record B which has already an item - an order from a record C used in a subscription - an order from a record D used in an other order 2. note the biblionumbers of the records used (or open them in other tabs in yout browser) 3. click on "Delete basket" 4. choose button "Delete basket and orders" 5. check the catalogue : records A,B,C,D must still be there Test B: 1. create a basket with 4 orders: - an order from a new record A - an order from a record B which has already an item - an order from a record C used in a subscription - an order from a record D used in an other order 2. note the biblionumbers of the records used (or open them in other tabs in yout browser) 3. click on "Delete basket" 4. choose button "Delete basket, orders and records" 5. check the catalogue : records B,C,D must still be there. Record A must be deleted Signed-off-by: Cedric Vita Signed-off-by: Chris Cormack -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #31 from mathieu saby --- Thank you, I did not know the trick. Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #30 from Jonathan Druart --- (In reply to mathieu saby from comment #29) > Hello Jonathan > Did you do something with this attachment? What was the issue ? Mathieu, I applied the third patch using 'git apply --reject' and I manually fix the conflict. -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #29 from mathieu saby --- Hello Jonathan Did you do something with this attachment? What was the issue ? Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791
Jonathan Druart changed:
What|Removed |Added
Attachment #21969|0 |1
is obsolete||
--- Comment #28 from Jonathan Druart ---
Created attachment 22046
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22046&action=edit
Bug 7791 Followup: add warning after deletion if some records were not deleted
This followup add some warnings after deletion if some items were not deleted.
4 types of messages are possible :
- x item(s) attached.
- x subscription(s) attached.
- x order(s) attached.
- Unknown error.
To test :
test a
1. create a basket with
- an order using a record A which has already an item, which is used in a
subscription, and in other order (in an other basket)
- an order using a record B used nowhere elese
2. click on "Delete basket"
3. choose button "Delete basket, orders and records"
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item, subscription and order.
5. check the link around the title of record B takes you to the record
6. check the link under the warning box ("Click here to go back to booksellers
page") takes you to booksellers page
5. check record B is deleted
test b
1. suppress the subscription linked with record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item and order
test c
1. suppress the item attached under record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its orderBug 7791 Followup: add warning after deletion if
some
records were not deleted
Content-Type: text/plain; charset="utf-8"
This followup add some warnings after deletion if some items were not deleted.
4 types of messages are possible :
- x item(s) attached.
- x subscription(s) attached.
- x order(s) attached.
- Unknown error.
To test :
test a
1. create a basket with
- an order using a record A which has already an item, which is used in a
subscription, and in other order (in an other basket)
- an order using a record B used nowhere elese
2. click on "Delete basket"
3. choose button "Delete basket, orders and records"
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item, subscription and order.
5. check the link around the title of record B takes you to the record
6. check the link under the warning box ("Click here to go back to booksellers
page") takes you to booksellers page
5. check record B is deleted
test b
1. suppress the subscription linked with record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item and order
test c
1. suppress the item attached under record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its order
--
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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 Jonathan Druart 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #27 from mathieu saby --- This bug applies correctly on current master in my VM : $git bz apply 7791 Bug 7791 - Deleting basket should not delete all existing orders without warning Bug 7791 : Give the ability to delete records when deleting a basket Apply? [yn] y Applying: Bug 7791 : Give the ability to delete records when deleting a basket Bug 7791 Followup: delete warns, add missing curly braket and make a simpler message if basket is void Apply? [yn] y Applying: Bug 7791 Followup: delete warns, add missing curly braket and make a simpler message if basket is void [PATCH 3/3] Bug 7791 Followup: add warning after deletion if some records were not deleted Apply? [yn] y Applying: Bug 7791 Followup: add warning after deletion if some records were not deleted Maybe the problem comes from the sandbox ? Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #26 from mathieu saby --- I could mabye try to merge the 3 patches, maybe it will resolve the issue. But I would not prefer, as the first 2 patches were already signed off, and not the 3rd... Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #25 from mathieu saby --- Same error Jonathan pointed out. I regenerated the patch and it applies cleany on my machine (using git am -3iu) Sorry, I don't understand this sha1 issue, and can't resolve it :( Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 Pierre Angot changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply --- Comment #24 from Pierre Angot --- The sandbox you've requested is not ready. Some problems occurred applying patches from bug 7791: Something went wrong !Applying: Bug 7791 : Give the ability to delete records when deleting a basket Applying: Bug 7791 Followup: delete warns, add missing curly braket and make a simpler message if basket is void Applying: Bug 7791 Followup: add warning after deletion if some records were not deleted fatal: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Patch failed at 0001 Bug 7791 Followup: add warning after deletion if some records were not deleted When you have resolved this problem run git bz apply --continue. If you would prefer to skip this patch, instead run git bz apply --skip. To restore the original branch and stop patching run git bz apply --abort. Bug 7791 - Deleting basket should not delete all existing orders without warning 21245 - Bug 7791 : Give the ability to delete records when deleting a basket 21246 - Bug 7791 Followup: delete warns, add missing curly braket and make a simpler message if basket is void 21969 - [PATCH 3/3] Bug 7791 Followup: add warning after deletion if some records were not deleted -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 mathieu saby changed: What|Removed |Added Attachment #21684|0 |1 is obsolete|| --- Comment #23 from mathieu saby --- Created attachment 21969 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21969&action=edit [PATCH 3/3] Bug 7791 Followup: add warning after deletion if some records were not deleted I applied it, and regenerate a patch with format pach. Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 mathieu saby 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #22 from Jonathan Druart --- (In reply to mathieu saby from comment #21) The last one. -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #21 from mathieu saby --- strange Do you know what attachment is causing the problem? Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #20 from Jonathan Druart --- Tested again, I cannot apply your patch, sorry. -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #19 from mathieu saby --- Jonathan, I don't understand, I don't have any problem : $git bz apply 7791 Bug 7791 - Deleting basket should not delete all existing orders without warning Bug 7791 : Give the ability to delete records when deleting a basket Apply? [yn] y Applying: Bug 7791 : Give the ability to delete records when deleting a basket Bug 7791 Followup: delete warns, add missing curly braket and make a simpler message if basket is void Apply? [yn] y Applying: Bug 7791 Followup: delete warns, add missing curly braket and make a simpler message if basket is void [PATCH 3] Bug 7791 Followup: add warning after deletion if some records were not deleted Apply? [yn] y Applying: Bug 7791 Followup: add warning after deletion if some records were not deleted Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #18 from mathieu saby --- zut -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 Jonathan Druart changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply --- Comment #17 from Jonathan Druart --- fatal: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt). -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 I'm just a bot changed: What|Removed |Added CC||git...@bugs.koha-community. ||org When did the bot||2013-10-02 last check this|| --- Comment #16 from I'm just a bot --- Patch applied cleanly, go forth and 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #15 from mathieu saby --- Patch 1 and 2 were already 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 mathieu saby changed: What|Removed |Added Status|Failed QA |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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791
--- Comment #14 from mathieu saby ---
Created attachment 21684
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21684&action=edit
[PATCH 3] Bug 7791 Followup: add warning after deletion if some records were
not deleted
Here is the followup.
This followup add some warnings after deletion if some items were not deleted.
4 types of messages are possible :
- x item(s) attached.
- x subscription(s) attached.
- x order(s) attached.
- Unknown error.
To test :
test a
1. create a basket with
- an order using a record A which has already an item, which is used in a
subscription, and in other order (in an other basket)
- an order using a record B used nowhere elese
2. click on "Delete basket"
3. choose button "Delete basket, orders and records"
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item, subscription and order.
5. check the link around the title of record B takes you to the record
6. check the link under the warning box ("Click here to go back to booksellers
page") takes you to booksellers page
5. check record B is deleted
test b
1. suppress the subscription linked with record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its item and order
test c
1. suppress the item attached under record A
2. create an other basket using record A
3. delete the basket on the same way as for test a
4. you must see a page anouncing basket deletion, and that record A was not
deleted because of its order
M. Saby
--
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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #13 from mathieu saby --- (In reply to M. de Rooy from comment #12) > (In reply to mathieu saby from comment #11) > > To be clear : does it occur in master without my patch? > Hi Matthieu, > Yes it does. So nothing to do with your patch.. > > Note that if I am in a basket from some supplier and I delete it, I actually > would like to return to the basket list of that supplier instead of getting > a list of all suppliers. But that is outside the scope of this report. I filled a new bug for that 10978 ;-) Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #12 from M. de Rooy --- (In reply to mathieu saby from comment #11) > To be clear : does it occur in master without my patch? Hi Matthieu, Yes it does. So nothing to do with your patch.. Note that if I am in a basket from some supplier and I delete it, I actually would like to return to the basket list of that supplier instead of getting a list of all suppliers. But that is outside the scope of this 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #11 from mathieu saby --- (In reply to M. de Rooy from comment #10) > QA Comment: > This looks quite good to me. No complaints from qa tools. > I just have one question: I think it would be even more friendly to the user > if you could report the errors DelBiblio returns on refused deletions > (because of items not connected to orders). > At this time your code does not look at a possible return value of DelBiblio. > Would it be possible for you to add this in a followup? (But I do not think > this should block your patch..) Hi I will look if it is possible and add a followup. > > And I got struck by another problem when testing your patch, but it does not > depend on your changes but is connected to booksellers.pl. > If I delete a basket, the return page is booksellers.pl. Every time I get > the Firefox message: A script on this page may be busy, or it may have > stopped responding. You can stop the script now, or you can continue to see > if the script will complete. > Script: http://[servername]/intranet-tmpl/lib/jquery/jquery.js:4 > Just mentioning it here for completeness. To be clear : does it occur in master without my patch? Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 M. de Rooy changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #10 from M. de Rooy --- QA Comment: This looks quite good to me. No complaints from qa tools. I just have one question: I think it would be even more friendly to the user if you could report the errors DelBiblio returns on refused deletions (because of items not connected to orders). At this time your code does not look at a possible return value of DelBiblio. Would it be possible for you to add this in a followup? (But I do not think this should block your patch..) And I got struck by another problem when testing your patch, but it does not depend on your changes but is connected to booksellers.pl. If I delete a basket, the return page is booksellers.pl. Every time I get the Firefox message: A script on this page may be busy, or it may have stopped responding. You can stop the script now, or you can continue to see if the script will complete. Script: http://[servername]/intranet-tmpl/lib/jquery/jquery.js:4 Just mentioning it here for completeness. Changing the bug status for now. -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 M. de Rooy changed: What|Removed |Added CC||m.de.r...@rijksmuseum.nl QA Contact||m.de.r...@rijksmuseum.nl --- Comment #9 from M. de Rooy --- QA: Looking at this one now. -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 mathieu saby changed: What|Removed |Added 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #8 from Jonathan Druart --- Created attachment 21246 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21246&action=edit Bug 7791 Followup: delete warns, add missing curly braket and make a simpler message if basket is void This patch - delete warns - add a missing } - add a condition in template of avoiding asking to suppress orders or records if the basket is void To test : 1. Make the same tests as defined in test plan of main patch. It should behave the same way 2. Try to delete a basket with no records inside. You will only have a "Delete basket" button, with less warnings Signed-off-by: Cedric Vita -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 Jonathan Druart changed: What|Removed |Added Attachment #21224|0 |1 is obsolete|| Attachment #21225|0 |1 is obsolete|| --- Comment #7 from Jonathan Druart --- Created attachment 21245 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21245&action=edit Bug 7791 : Give the ability to delete records when deleting a basket Currently, when a basket is deleted, all the orders are deleted (there is a foreign key in aqorders table on basketno). This could be dangerous, and there is no warn. After the deletion, unused biblios are left in the catalogue. This patch - adds a more detailed message describing the consequences of deletion - give the choice of also deleting biblio records if possible To test : Test A : 1. create a basket with 4 orders: - an order from a new record A - an order from a record B which has already an item - an order from a record C used in a subscription - an order from a record D used in an other order 2. note the biblionumbers of the records used (or open them in other tabs in yout browser) 3. click on "Delete basket" 4. choose button "Delete basket and orders" 5. check the catalogue : records A,B,C,D must still be there Test B: 1. create a basket with 4 orders: - an order from a new record A - an order from a record B which has already an item - an order from a record C used in a subscription - an order from a record D used in an other order 2. note the biblionumbers of the records used (or open them in other tabs in yout browser) 3. click on "Delete basket" 4. choose button "Delete basket, orders and records" 5. check the catalogue : records B,C,D must still be there. Record A must be deleted Signed-off-by: Cedric Vita -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 Pierre Angot changed: What|Removed |Added Status|Needs Signoff |Signed Off CC||tredok.pie...@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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 cedric.v...@dracenie.com changed: What|Removed |Added CC||cedric.v...@dracenie.com --- Comment #6 from cedric.v...@dracenie.com --- Signing patches on sandbox (test1) did not work. Thank you to "Sign-off" on. -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #5 from mathieu saby --- Created attachment 21225 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21225&action=edit [PATCH 2] Bug 7791 Followup: delete warns, add missing curly braket and make a simpler message if basket is void This patch - delete warns - add a missing } - add a condition in template of avoiding asking to suppress orders or records if the basket is void To test : 1. Make the same tests as defined in test plan of main patch. It should behave the same way 2. Try to delete a basket with no records inside. You will only have a "Delete basket" button, with less warnings -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #4 from mathieu saby --- Oups, I was not suppose to keep warns. It was just for me... followup coming Mathieu -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 Jonathan Druart changed: What|Removed |Added CC||jonathan.dru...@biblibre.co ||m --- Comment #3 from Jonathan Druart --- Mathieu, Some warns are in french and some indentations should be 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 --- Comment #2 from mathieu saby --- Created attachment 21224 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21224&action=edit Bug 7791 : Give the ability to delete records when deleting a basket Currently, when a basket is deleted, all the orders are deleted (there is a foreign key in aqorders table on basketno). This could be dangerous, and there is no warn. After the deletion, unused biblios are left in the catalogue. This patch - adds a more detailed message describing the consequences of deletion - give the choice of also deleting biblio records if possible To test : Test A : 1. create a basket with 4 orders: - an order from a new record A - an order from a record B which has already an item - an order from a record C used in a subscription - an order from a record D used in an other order 2. note the biblionumbers of the records used (or open them in other tabs in yout browser) 3. click on "Delete basket" 4. choose button "Delete basket and orders" 5. check the catalogue : records A,B,C,D must still be there Test B: 1. create a basket with 4 orders: - an order from a new record A - an order from a record B which has already an item - an order from a record C used in a subscription - an order from a record D used in an other order 2. note the biblionumbers of the records used (or open them in other tabs in yout browser) 3. click on "Delete basket" 4. choose button "Delete basket, orders and records" 5. check the catalogue : records B,C,D must still be there. Record A must be deleted -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 mathieu saby changed: What|Removed |Added Status|ASSIGNED|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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 mathieu saby changed: What|Removed |Added Status|NEW |ASSIGNED -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 mathieu saby changed: What|Removed |Added Assignee|henridam...@koha-fr.org |mathieu.saby@univ-rennes2.f ||r --- Comment #1 from mathieu saby --- Patch coming soon M. Saby -- 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 7791] Deleting basket should not delete all existing orders without warning
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791 mathieu saby changed: What|Removed |Added CC||mathieu.saby@univ-rennes2.f ||r -- 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/
