DCLP / dclpxsltbox

Sandbox for development, testing, and review of XSLT for DCLP
http://dclp.github.io/dclpxsltbox/
1 stars 5 forks source link

CWKB numbers not being deleted #273

Closed rla2118 closed 7 years ago

rla2118 commented 7 years ago

Residual CWKB numbers are not being deleted from the XML whenever an attempt is made to save a new number.

paregorios commented 7 years ago

This sounds like a problem with the editor to me. Is that your impression @rla2118 @jcowey @Edelweiss? I've added the "editor" label based on that assumption.

Edelweiss commented 7 years ago

In DCLP EpiDoc there is no specific xpath reserved for CWKB numbers (such as a TEI:idno).

The representation of CWKB numbers within EpiDoc is a URL inside a @ref attribute:

<author ref="http://cwkb.org/author/id/1234/rdf http://data.perseus.org/catalog/urn:cts:greekLit:tlg2345 http://data.perseus.org/catalog/urn:cts:latinLit:phi7654 http://catalog.perseus.org/catalog/urn:cts:latinLit:stoa6789"/>

If you want to get rid of a CWKB number, you also have to remove the corresponding link.

If you don’t remove the URL, the editor will (for your convenience) restore the CWKB number from the given URL the next time you call up that file for display in the editor mask.

(The same goes for TLG, Stoa and PHI numbers)

bildschirmfoto 2017-08-09 um 11 53 17

We could change that behaviour so that the editor will delete any URL if there is an empty string in the corresponding number field. But beware: If we use this logic, the users will have to provide a CWKB number whenever they want to save a link to CWKB. If they only provide a link and leave the corresponding number field blank, the link will be deleted. Let’s call that alternative (I).

Further alternatives could be:

(II) Make the editor delete links automatically once the number in the corresponding text field is erased (and of course the other way around, i.e. when a link is deleted, the corrsponding number will disappear). Perhaps a message box could inform the users about the repercussions of their actions. This would be the client side version of alternative (I).

(III) Hide the links (as suggested during one of our hands-on workshops here in Heidelberg). It’s actually very much like alternative (I), but the users won’t see that there are any links unless they view the xml. This might, however, make them think that they need to provide the links themselves. SoSOL can, of course, remove dupliactes, but the users might wonder why the data they enter is not being displayed in the editor mask. The entry field would look like this:

iii

(IV) Get rid of the numbers at the top altogether to have the editor reflect the underlying xml as accurately as possible. It would look like this:

iv

I would opt for alternatives (II) or (IV) or leave it as it is, as these are the alternatives with either the least suprise or the least chances of accidental data loss.

paregorios commented 7 years ago

I will ask the users here at the workshop in NYC today which approach they would prefer.

Edelweiss commented 7 years ago

Cool!

paregorios commented 7 years ago

Users here would prefer not to see the URLs at all. Further, they recommend an interface that would look like this:

dclp-authors-works

Edelweiss commented 7 years ago

a few example tei:author/@ref that we currently have in the system:

Edelweiss commented 7 years ago

examples for tei:title/@ref

paregorios commented 7 years ago

@Edelweiss where should we look to test this? cc @jcowey

paregorios commented 7 years ago

per discussion with @jcowey @Edelweiss @m-k-r this morning, Heidelberg to provide links to committed code discussed above

Edelweiss commented 7 years ago

This can be tested on litpap.info. Testing was meant to ensure that the new interface didn’t break the underlying logic that is responsible for loading from and saving to EpiDoc.

As this was lower priority the desiderata were not fully implemented.

Yet to be done:

Edelweiss commented 7 years ago

I listed only the relevant commits. There are also a few other commits that are related to the same editor component but are not part of the resolution of this issue. Regarding the commits mentioned above in an isolated container may cause some quirks, though, such as wrong captions for digital identifiers.

Edelweiss commented 7 years ago

The part of this ticket that concerns the new user interface lives on in #331