internetarchive/openlibrary

Edition's bookcover alt-text should pull author from work

Open

#486 geöffnet am 28. Apr. 2017

Auf GitHub ansehen
 (26 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Python (4.626 Stars) (1.172 Forks)batch import
Affects: DataAffects: ServerAffects: UIGood First IssueLead: @cdriniNeeds: ResponsePriority: 3Type: Bug

Beschreibung

Correcting a work's author (e.g. https://openlibrary.org/works/OL3531880W/Living_with_Leviathan?b=4&a=3&_compare=Compare&m=diff ) also corrects the Alt-text for the work cover, (e.g. to alt="Cover of: Living with Leviathan by David Brian Smith" ) but fails to correct the Alt-text for the editions below it, instead leaving it untouched (e.g. as alt="Cover of: Living with Leviathan by David Buchanan Smith" ). Might this be why the edition is still found on searches for the old author spelling?

view-source:https://openlibrary.org/books/OL17631357M/Living_with_Leviathan shows the error at lines 349, 357, 371

Proposed fix: Ensure the edition alt-text author name uses the same source data as the rest of the edition interface, which takes the author attached to the work(if it exists)

Contributor Guide