Bug 11967 - Add support for RMA notices to Acquisitions.
Summary: Add support for RMA notices to Acquisitions.
Status: RESOLVED WONTFIX
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-03-19 20:50 UTC by Frédérick Capovilla
Modified: 2023-07-09 10:39 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Medium patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Add RMA support for Acquisitions. (22.97 KB, patch)
2014-03-19 20:50 UTC, Frédérick Capovilla
Details | Diff | Splinter Review
Bug 11967 - Add support for RMA notices to Acquisitions. (22.00 KB, patch)
2014-09-04 19:17 UTC, simith.doliveira
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Frédérick Capovilla 2014-03-19 20:50:43 UTC Comment hidden (obsolete)
Comment 1 Paola Rossi 2014-06-18 15:55:40 UTC
I've tried to apply against 3.17.00.008 :

Applying: Add RMA support for Acquisitions.
Using index info to reconstruct a base tree...
<stdin>:31: trailing whitespace.
        "select aqorders.*,aqbasket.*,biblio.*,biblioitems.* from aqorders 
<stdin>:32: trailing whitespace.
        LEFT JOIN aqbasket on aqbasket.basketno=aqorders.basketno 
<stdin>:33: trailing whitespace.
        LEFT JOIN biblio on aqorders.biblionumber=biblio.biblionumber 
<stdin>:34: trailing whitespace.
        LEFT JOIN biblioitems on aqorders.biblioitemnumber=biblioitems.biblioitemnumber 
<stdin>:333: trailing whitespace.
Certains items de la commande seront retournés : 
warning: squelched 2 whitespace errors
warning: 7 lines add whitespace errors.
Falling back to patching base and 3-way merge...
Auto-merging tools/letter.pl
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
Auto-merging koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
Auto-merging installer/data/mysql/updatedatabase.pl
CONFLICT (content): Merge conflict in installer/data/mysql/updatedatabase.pl
Auto-merging installer/data/mysql/fr-FR/1-Obligatoire/sample_notices.sql
CONFLICT (content): Merge conflict in installer/data/mysql/fr-FR/1-Obligatoire/sample_notices.sql
Auto-merging installer/data/mysql/en/mandatory/sample_notices.sql
CONFLICT (content): Merge conflict in installer/data/mysql/en/mandatory/sample_notices.sql
Auto-merging C4/Letters.pm
Failed to merge in the changes.
Patch failed at 0001 Add RMA support for Acquisitions.

So I pass the patch to "Patch doesn't apply" status.
Comment 2 simith.doliveira 2014-09-04 19:17:48 UTC
Created attachment 31381 [details] [review]
Bug 11967 - Add support for RMA notices to Acquisitions.
Comment 3 Paola Rossi 2014-09-09 14:29:36 UTC
I've applied against master 3.17.00.022 

1) On the acquisitions section, I clicked on the "Return order" menu item.
"No baskets for this vendor" was shown, whilst there was no current vendor.
2) I asked for a non existent vendor, and Koha answered me "No baskets for this vendor".
3) I asked for a vendor without baskets. Instead of "No baskets for this vendor" Koha sent, beneath the empty result table, the possibility of selecting an "Output" and of clicking "Generate RMA".
4) I asked for a vendor without orders. Instead of a message like "No orders for this vendor" Koha sent, beneath the empty result table, the possibility of selecting an "Output" and of clicking "Generate RMA". 
5) On tools I copied the RMA notice to my library Centerville. Then I asked for a vendor with a basket. "Using Notice:" should have listed two options, but there was only one, the "All libraries" notice.
6) I selected some orders and then asked to "Send mail", but the vendor received no mail. 
7) After selecting "Generate RMA", no message of feedback was sent: "sent mail"/"problems occurred ..." (see bug 5342).
8) Koha listed the partially received orders too, but perhaps this could be an error.  
In fact: RMA = "Returning a product in order to receive a refund, replacement, or repair during the product's warranty period". Perhaps RMA should concern only products already received. I asked for a vendor with some partially received orders: these orders were listed. Is it right?
9) In the case of partially received orders, in the result table the row for the received order is exactly the same as the row for the partial received order. So the two orders are not distinct.

So I pass the patch to "Failed QA" status.
Comment 4 Mark Tompsett 2015-05-09 15:15:23 UTC
More reasons to fail qa.
This only added it to english and french. This must be added to all sample notices. MY next concern is there are no tests. When you modify a C4 or Koha library, you must have test coverage. The test file t/db_dependent/Letters.t was not modified to include such coverage.
Comment 5 Blou 2015-07-31 12:52:49 UTC
Please close this bz.  I would do it but this is not an option I have.

The sponsor is not interested in pursuing it.  We'll put our energies on other bz.
Comment 6 Katrin Fischer 2015-08-02 17:17:35 UTC
Hi Blou, I have removed the assigned status and the 'sponsored'. 

I am hesitating to close this, as it might be a feature someone else might want to work at.
Comment 7 Katrin Fischer 2023-07-09 10:39:55 UTC
(In reply to Blou from comment #5)
> Please close this bz.  I would do it but this is not an option I have.
> 
> The sponsor is not interested in pursuing it.  We'll put our energies on
> other bz.

(In reply to Katrin Fischer from comment #6)
> Hi Blou, I have removed the assigned status and the 'sponsored'. 
> 
> I am hesitating to close this, as it might be a feature someone else might
> want to work at.

8 years... no longer hesitating. Code and specifications would probably need a fresh start with all changes since.