faustedition / faust-gen-html

Pipelines to generate HTML for the Faust edition's reading texts and prints.
1 stars 4 forks source link

App entries displaced when occuring in the same chunk #614

Open gerritbruening opened 5 years ago

gerritbruening commented 5 years ago
gerritbruening commented 5 years ago

Faust II Titel ist analog, aber richtig.

thvitt commented 5 years ago

7729 ist auch richtig, oder?

thvitt commented 5 years ago

Das Problem mit der Hexe ist folgendes. Apparateintrag:

    <app>
        <f:ins n="before_2465_a" place="before"><move type="entrance"/></f:ins>
        <f:replace n="before_2465_a">Die Hexe</f:replace>
        <f:ins n="before_2465_a">Die Hexe</f:ins>
        <ref>vor 2465</ref>
        <lem/>
        <rdg wit="A B C.1_12 C.3_12" type="type_1c">Die Hexe <note>wird in <wit>A</wit>
                <wit>B</wit> <wit>C.1_12</wit> <wit>C.3_12</wit> nicht als Auftritt markiert (S
                unterscheidet typographisch nicht zwischen Auftritten und
            Sprecherangaben).</note></rdg>
    </app>

Da hier mehrere XML-Bearbeitungsschritte (f:ins) gemeinsam vorkommen (Einfügen des <move> und Markieren des Die Hexe) werden zunächst auch zwei Kopien des Apparateintrags eingefügt, eine vor der BA (wg. before) und eine darin (wg. Die Hexe). Im Aufräumschritt wird dann die zweite weggeräumt und es bleibt die erste stehen.

thvitt commented 5 years ago

Dieser Fall scheint lt. Suche in den Apparatdateien ausschließlich an dieser Stelle (Hexe, before_2465_a) vorzukommen.

gerritbruening commented 5 years ago

7729 ist auch richtig, oder?

Ja, jetzt ja, seltsam. Ich habs wirklich gesehen ...

thvitt commented 5 years ago

In other cases, a mixup might occur when there are two app entries for the same line that appear in the wrong order in (app1norm.xml, app2norm.xml, app12norm-special-cases.xml). This might have been inadvertedly fixed in d4bd311618531e78e070b4d4702a858b451237c7 for 7729.

thvitt commented 5 years ago

Here's a list of lines that contain multiple apparatus entries and are thus candidates for confusion:

thvitt commented 5 years ago
gerritbruening commented 5 years ago

In the referencing commit I have reordered the entries both pointing to before_2270_a.