While a librarian is cataloguing an authority record by using the framework cannot check the full view of the record in MARC structure way before saving. That patch will add a new action on edit mode in order to show on a new window a preview of MARC record that will be saved.
George is this something you're working on? If so please assign it to yourself and mark the bug ASSIGNED.
Yes you are right, I just did that.
Created attachment 140063 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing.
Template process failed: file error - modals/edit_authority_preview.inc: not found at /kohadevbox/koha/C4/Templates.pm line 127 Looks like you may have forgotten to 'git add' the new include file?
Created attachment 140115 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing.
Created attachment 140180 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing.
Created attachment 140181 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing.
(In reply to Owen Leonard from comment #4) > Template process failed: file error - modals/edit_authority_preview.inc: not > found at /kohadevbox/koha/C4/Templates.pm line 127 > > Looks like you may have forgotten to 'git add' the new include file? I think now I did in right way thanks to Andreas
Created attachment 140182 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing.
Created attachment 140183 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing.
Created attachment 140184 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing. Mentored by: Andreas Roussos
Created attachment 140185 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing. Mentored by: Andreas Roussos
Created attachment 140186 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view A new file for modal view of result added.
(In reply to Owen Leonard from comment #4) > Template process failed: file error - modals/edit_authority_preview.inc: not > found at /kohadevbox/koha/C4/Templates.pm line 127 > > Looks like you may have forgotten to 'git add' the new include file? Ok now it looks right, sorry for multiple tries but now you can test it in a normal way.
Hi, after applying the patch : - the Edit option is in the 'Action' drop down menu - when I click on this option, the record is displayed but the screen keeps on loading and, even if I see the 'Marc preview' button on the toolbar, I can't do anything (before applying the patch, the 'MARC preview' option in the 'Actions' drop down menu display a pop up which is blank)
Created attachment 152915 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing. Mentored by: Andreas Roussos
Created attachment 152916 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view A new file for modal view of result added.
Created attachment 152917 [details] [review] Bug 29825: (follow-up) Some style fixes and modal correction This patch makes a few coding style changes to the new JavaScript and updates the Bootstrap modal markup to be in line with current standards. The previous patches lacked a trigger for the preview modal, so I added that. Since the content of the modal uses spaces for formatting I added monospace formatting to the modal body.
I was curious about this so I thought I'd rebase it and make a few minor corrections just to get it back in play.
Using two spaces between the subfield label and the subfield value is jarring. One space isn't entirely shocking, detail.pl does that, or given that this looks more like the advanced bib editor than detail.pl no spaces but some padding or whatever that uses to get a little separation without a whole space wouldn't be shocking to advanced editor users, but two spaces looks like I accidentally started every field with a leading space. It would also be vastly more useful (and probably more accessible) if it copied over the title attributes from tagnum and subfieldcode and used them as titles in the preview.
But when I went looking for our house style for displaying MARC, I see that we use either nothing or a leading _ or ‡ or a trailing {space}/ as a subfield delimiter, that we either make the subfield code bold or do not, and that we either follow it with a space or do not, and that we either put each subfield on a new line or do not. _aTitle and <strong>_a</strong> Title are even just two views from the same script. Maybe our house style is "every MARC display must look different than any other MARC display" :)
Created attachment 153078 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing. Mentored by: Andreas Roussos Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org>
Created attachment 153079 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view A new file for modal view of result added. Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org>
Created attachment 153080 [details] [review] Bug 29825: (follow-up) Some style fixes and modal correction This patch makes a few coding style changes to the new JavaScript and updates the Bootstrap modal markup to be in line with current standards. The previous patches lacked a trigger for the preview modal, so I added that. Since the content of the modal uses spaces for formatting I added monospace formatting to the modal body. Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org>
Please fix commit names ! First two patches have the same name, different contents.
The commit name for the second would have been "Bug 29825: Oops, I forgot to 'git add'" so really rather than fixing it, the first two should just be squashed.
Ok great, so you can do it, attach and the ticket will sort itself out automatically I think (will only keep your two patches after the squash). Otherwise it will need to be manually obsoleted ("Details" link next to the patch)
Created attachment 159200 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing. Mentored by: Andreas Roussos Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org>
Created attachment 159201 [details] [review] Bug 29825: (follow-up) Some style fixes and modal correction This patch makes a few coding style changes to the new JavaScript and updates the Bootstrap modal markup to be in line with current standards. The previous patches lacked a trigger for the preview modal, so I added that. Since the content of the modal uses spaces for formatting I added monospace formatting to the modal body. Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org>
Created attachment 164126 [details] [review] Bug 29825: Preview of authority record on edit mode as MARC formatted view This bug adds a new functionality on Edit mode of a new or existing authority. Add a new button 'MARC Preview' in order librarian have the ability to preview the record on MARC plain structure mode and have all fields and subfields that entered before save/update the record. To test: 1. Open or add an authority 2. The preview button isn't on toolbar 3. Apply the patch 4. Do step 1 5. The preview button exists 6. Do some changes or not to the authority record 7. Click to Preview MARC button and a modal window will appear with all fields indicators and subfields that filled in the form of cataloguing. Mentored by: Andreas Roussos Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 164127 [details] [review] Bug 29825: (follow-up) Some style fixes and modal correction This patch makes a few coding style changes to the new JavaScript and updates the Bootstrap modal markup to be in line with current standards. The previous patches lacked a trigger for the preview modal, so I added that. Since the content of the modal uses spaces for formatting I added monospace formatting to the modal body. Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
This patch set contains some small issues I have fixed in a follow-up. Please keep in mind for later submissions :) This string in JavaScript is not translatable. _() must be used: + var content = "<h3>Authority #: [% authid | html %]</h3><hr>"; Also capitalization should be "MARC preview" according to our guidelines here: + <a class="btn btn-default" id="preview" href="#"><i class="fa fa-eye"></i> MARC Preview</a> One general question: With this we have a different style of preview from the result list of the authortiy search and the detail page. Should they be labelled differently? Should they be styled the same?
Pushed for 24.05! Well done everyone, thank you!
(In reply to Katrin Fischer from comment #32) > With this we have a different style of preview from > the result list of the authority search and the detail page. I think the preview from the authority search result page should be replaced with this version. It's more consistent with the way we show MARC previews from the bibliographic detail page.
Filed separately: Bug 36711 - Replace "MARC preview" in authority results with display from bug 29825