search

kitodo / kitodo-ugh

Kitodo.UGH Library
2 stars 10 forks source link

Merge: [ZM] Generieren von Supraankern (#59) #1

Closed matthias-ronge closed 9 years ago

matthias-ronge commented 10 years ago

Die UGHlib musste musste erweitert werden: Anstelle genau einer Ankerdatei, welche genau die Metadaten der oberste Ebene des logischen Strukturbaumes enthält, können nun beliebig viele Ankerdateien, die jeweils beliebig viele Ebenen des logischen Strukturbaumes enthalten können, konfiguriert werden.

Im Regelsatz kann dazu anstelle von anchor="true" ein beliebiger anchor="anchorClassName" verwendet werden (wobei true weiterhin eine gültige Möglichkeit für den Namen ist). Um mehrere hierarchisch aufeinander folgende DocStrctType-Elemente in der selben Ankerdatei zu speichern, muss ihnen der gleiche Name gegeben werden, wobei der Name der geschriebenen Ankerdatei dem vergebenen Namen entspricht. Ausnahme: true resultiert weiterhin in anchor.

henning-gerhardt commented 10 years ago

Changes contains mixed tab / space indents in files:

ugh/src/ugh/dl/DigitalDocument.java ugh/src/ugh/dl/DocStruct.java ugh/src/ugh/dl/DocStructType.java ugh/src/ugh/dl/Prefs.java ugh/src/ugh/fileformats/mets/MetsMods.java ugh/src/ugh/fileformats/mets/MetsModsImportExport.java

All mentioned above files use space indent but your changed / added lines of code contain tab indent. Other changed files are use tab indent.

matthias-ronge commented 10 years ago

Leitfaden für Goobi-Entwickler states „Einrückungen erfolgen mit Tabs“, so—if I got it right—what you describe is what I would expect to be the case. Of course, I tried not to touch anything I didn’t actually change, so other lines and files will remain whatever they were before.

henning-gerhardt commented 10 years ago

You are right.

If this merge will be done then we had mixed indents in this files. Files are created long before Goobi Coding Guidelines are written and my opinion about this is: use this indent where the whole file is and don't mix different indents in a file.

Right way should be: format project according current Goobi Coding Guidelines.

sebastian-meyer commented 10 years ago

Both of you are right: there shouldn't be different types of indention in one file and the coding guidelines demand to use tabs instead of spaces. Thus I would suggest changing all indention to tabs in the whole file.