Bug 17963

Summary: TT syntax for notices - Prove that AR_* are compatible
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: enhancement    
Priority: P5 - low CC: josef.moravec, katrin.fischer, kyle, m.de.rooy, veron
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 17959, 17962    
Bug Blocks: 17961, 17964    
Attachments: Bug 17963: TT syntax for notices - Prove that AR_* are compatible
Bug 17963: TT syntax for notices - Prove that AR_* are compatible
Bug 17963: TT syntax for notices - Prove that AR_* are compatible
Bug 17963: TT syntax for notices - Prove that AR_* are compatible
Bug 17963: TT syntax for notices - Prove that AR_* are compatible
Bug 17963: TT syntax for notices - Prove that AR_* are compatible

Description Jonathan Druart 2017-01-23 13:11:50 UTC

    
Comment 1 Jonathan Druart 2017-01-23 15:52:30 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2017-01-23 15:54:25 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2017-01-23 16:44:43 UTC Comment hidden (obsolete)
Comment 4 Josef Moravec 2017-01-24 19:47:55 UTC
I think there is missing article_requests table definition in C4::Letters, I am getting the notice generated without any information from article_requests table... but test is passing...
Comment 5 Jonathan Druart 2017-01-25 09:00:19 UTC
Created attachment 59539 [details] [review]
Bug 17963: TT syntax for notices - Prove that AR_* are compatible

Nothing new here since bug 17962, the AR_* notice messages are quite
simple. They send the article_request, patron, biblio, biblioitem, item and
library linked to the article request.

All the fields from these 6 tables should still be accessible using the
TT syntax.

Test plan:
Define TT notice templates for AR_PENDING, AR_PROCESSING, AR_COMPLETED
or AR_CANCELED.

You should manage to create a template to generate the same result as
the historical syntax.
Comment 6 Jonathan Druart 2017-01-25 09:01:04 UTC
(In reply to Josef Moravec from comment #4)
> I think there is missing article_requests table definition in C4::Letters, I
> am getting the notice generated without any information from
> article_requests table... but test is passing...

Yes indeed, the test was completely wrong!
Comment 7 Kyle M Hall 2017-02-17 11:51:26 UTC
Created attachment 60402 [details] [review]
Bug 17963: TT syntax for notices - Prove that AR_* are compatible

Nothing new here since bug 17962, the AR_* notice messages are quite
simple. They send the article_request, patron, biblio, biblioitem, item and
library linked to the article request.

All the fields from these 6 tables should still be accessible using the
TT syntax.

Test plan:
Define TT notice templates for AR_PENDING, AR_PROCESSING, AR_COMPLETED
or AR_CANCELED.

You should manage to create a template to generate the same result as
the historical syntax.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 8 Marcel de Rooy 2017-03-24 07:49:39 UTC
QA: Looking here now
Comment 9 Marcel de Rooy 2017-03-24 07:57:10 UTC
Created attachment 61594 [details] [review]
Bug 17963: TT syntax for notices - Prove that AR_* are compatible

Nothing new here since bug 17962, the AR_* notice messages are quite
simple. They send the article_request, patron, biblio, biblioitem, item and
library linked to the article request.

All the fields from these 6 tables should still be accessible using the
TT syntax.

Test plan:
Define TT notice templates for AR_PENDING, AR_PROCESSING, AR_COMPLETED
or AR_CANCELED.

You should manage to create a template to generate the same result as
the historical syntax.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 10 Kyle M Hall 2017-03-31 11:03:24 UTC
Pushed to master for 17.05, thanks Jonathan!
Comment 11 Katrin Fischer 2017-04-02 16:20:22 UTC
Dependencies are not in 16.11.x, so this is not needed there.