search

sciencehistory / chf-sufia

sufia-based hydra app
Other
9 stars 4 forks source link

Additional title doesn't seem to be alpha sorting #527

Closed catlu closed 7 years ago

catlu commented 7 years ago

Example: https://hydra.chemheritage.org/concern/generic_works/ft848q626

jrochkind commented 7 years ago

@hackmastera can you give me any hints about what code in what general area is supposed to alpha sort multiple entries?

hackartisan commented 7 years ago

For fields listed in the metadata table, it happens at the renderer layer. Here's the commit where I introduced it (knowing more now about reopening classes I would not have copied over this whole file but that is what I did at the time) https://github.com/chemheritage/chf-sufia/commit/6e7da7d91ec1d4060eec14113ee2bf59bfdd96c8#diff-ccbf0cdadb293f7b347b432b7b33fdb2R29

However in this case it was pulled out of the metadata table to be up below the title; there is no existing logic that I know of that should be used.