Merge pull request #1296 from digitaljudaica/cleanup

Cleanup
opentorah · Feb 24, 2020 · eaf4070 · eaf4070
2 parents a1eda9c + 53953a4
commit eaf4070
Show file tree

Hide file tree

Showing 12 changed files with 60 additions and 70 deletions.
diff --git a/collector/src/main/scala/org/digitaljudaica/archive/collector/Errors.scala b/collector/src/main/scala/org/digitaljudaica/archive/collector/Errors.scala
diff --git a/collector/src/main/scala/org/digitaljudaica/archive/collector/Main.scala b/collector/src/main/scala/org/digitaljudaica/archive/collector/Main.scala
@@ -1,7 +1,7 @@
 package org.digitaljudaica.archive.collector
 
 import java.io.File
-import org.digitaljudaica.xml.Print
+import org.digitaljudaica.xml.{From, Parser, Print, Xml}
 import org.digitaljudaica.archive.collector.reference.Names
 import scala.xml.{Elem, Text}
 
@@ -29,9 +29,16 @@ object Main {
     writeCollectionsTree(collectionsSorted, layout)
     writeIndex(collectionsSorted, layout)
 
+    println("Reading names.")
+    val names: Names =
+      Parser.parseDo(From.file(layout.docs, layout.namesListsFileName).parse(
+        Xml.withName("names", Names.parser(layout.namesDirectory, layout))))
+
     println("Processing name references.")
-    val names: Names = Names(layout.namesDirectory, layout)
-    names.processReferences(collections.flatMap(_.references))
+    names.addDocumentReferences(collections.flatMap(_.references))
+    names.checkReferences()
+    names.writeNames(layout.namesDirectory)
+    names.writeList(layout.namesFileDirectory, layout.namesFileName, layout)
   }
 
   private def writeIndex(collections: Seq[Collection], layout: Layout): Unit = Util.writeTei(

diff --git a/collector/src/main/scala/org/digitaljudaica/archive/collector/reference/Names.scala b/collector/src/main/scala/org/digitaljudaica/archive/collector/reference/Names.scala
@@ -1,15 +1,13 @@
 package org.digitaljudaica.archive.collector.reference
 
 import java.io.File
-import org.digitaljudaica.archive.collector.{CollectionLike, Errors, Layout, Util}
+import org.digitaljudaica.archive.collector.{CollectionLike, Layout, Util}
 import org.digitaljudaica.xml.{ContentType, Error, From, Parser, Xml}
 import org.digitaljudaica.util.Files
 import zio.{IO, ZIO}
-
 import scala.xml.{Elem, Text}
 
 final class Names private(
-  directory: File,
   layout: Layout,
   override val reference: String,
   teiNameds: Seq[org.digitaljudaica.reference.Named],
@@ -20,25 +18,28 @@ final class Names private(
 
   private val lists: Seq[NamesList] = listDescriptors.map(_.fillOut(nameds))
 
-  private val references: Seq[Reference] = nameds.flatMap(_.references)
-
   def findByRef(ref: String): Option[Named] = nameds.find(_.id == ref)
 
-  def processReferences(documentReferences: Seq[Reference]): Unit = {
-    val errors: Errors = new Errors
-    val references: Seq[Reference] = (this.references ++ documentReferences).filterNot(_.name == Text("?"))
-    for (reference <- references) reference.check(this, errors)
-    errors.check()
+  private var references: Seq[Reference] = null
 
-    // Individual names
-    for (named <- nameds) Util.writeTei(
-      directory,
-      fileName = named.id,
-      head = None,
-      content = named.toXml(references),
-      target = "namesViewer"
-    )
+  def addDocumentReferences(documentReferences: Seq[Reference]): Unit = {
+    references = (nameds.flatMap(_.references) ++ documentReferences).filterNot(_.name == Text("?"))
+  }
+
+  def checkReferences(): Unit = {
+    val errors: Seq[String] = references.flatMap(_.check(this))
+    if (errors.nonEmpty) throw new IllegalArgumentException(errors.mkString("\n"))
+  }
+
+  def writeNames(directory: File): Unit = for (named <- nameds) Util.writeTei(
+    directory,
+    fileName = named.id,
+    head = None,
+    content = named.toXml(references),
+    target = "namesViewer"
+  )
 
+  def writeList(directory: File, fileName: String, layout: Layout): Unit = {
     // List of all names
     val nonEmptyLists = lists.filterNot(_.isEmpty)
     val content: Seq[Elem] =
@@ -47,8 +48,8 @@ final class Names private(
         (for (list <- nonEmptyLists) yield list.toXml)
 
     Util.writeTei(
-      directory = layout.namesFileDirectory,
-      fileName = layout.namesFileName,
+      directory = directory,
+      fileName = fileName,
       head = Some(Text(reference)),
       content,
       target = "namesViewer"
@@ -58,11 +59,7 @@ final class Names private(
 
 object Names {
 
-  def apply(directory: File, layout: Layout): Names =
-    Parser.parseDo(From.file(layout.docs, layout.namesListsFileName).parse(
-      Xml.withName("names", parser(directory, layout))))
-
-  private def parser(
+  def parser(
     directory: File,
     layout: Layout,
   ): Parser[Names] = for {
@@ -77,7 +74,6 @@ object Names {
       if (errors.nonEmpty) IO.fail(errors.mkString("--", "\n--", "")) else IO.succeed(results)
     }
   } yield new Names(
-    directory,
     layout,
     reference,
     teiNameds,

diff --git a/collector/src/main/scala/org/digitaljudaica/archive/collector/reference/Reference.scala b/collector/src/main/scala/org/digitaljudaica/archive/collector/reference/Reference.scala
@@ -1,6 +1,5 @@
 package org.digitaljudaica.archive.collector.reference
 
-import org.digitaljudaica.archive.collector.Errors
 import org.digitaljudaica.reference.Entity
 import scala.xml.{Elem, Node}
 
@@ -16,11 +15,12 @@ final case class Reference(
   def role: Option[String] = reference.role
   def ref: Option[String] = reference.ref
 
-  def check(names: Names, errors: Errors): Unit = {
-    ref.fold(errors.error(s"Missing 'ref' attribute: Name>$name< ($source)")) { ref =>
-      if (ref.contains(" ")) errors.error(s"""Value of the ref attribute contains spaces: ref="$ref" """) else {
-        names.findByRef(ref).fold(errors.error(s"""Unresolvable reference: Name ref="$ref">${name.text}< """)) { named =>
-          if (named.entity != entity) errors.error(s"$entity reference to ${named.entity} ${named.name}: $name [$ref]")
+  def check(names: Names): Option[String] = {
+    ref.fold[Option[String]](Some(s"Missing 'ref' attribute: Name>$name< ($source)")) { ref =>
+      if (ref.contains(" ")) Some(s"""Value of the ref attribute contains spaces: ref="$ref" """) else {
+        names.findByRef(ref).fold[Option[String]](Some(s"""Unresolvable reference: Name ref="$ref">${name.text}< """)) { named =>
+          if (named.entity != entity) Some(s"$entity reference to ${named.entity} ${named.name}: $name [$ref]")
+          else None
         }
       }
     }

diff --git a/docs/collections/dubnov/tei/088.xml b/docs/collections/dubnov/tei/088.xml
@@ -173,7 +173,7 @@
         <item><label>№ 19.</label> <date when="1800-12-??">Декабрь 1800</date>. Реестръ бумагъ (14 №№) и книгъ (103 №№)
           <persName ref="alter-rebbe">Залмана Боруховича</persName>, съ переводомъ его двухъ молитвъ за царя, съ
           удостовѣреніемъ составителей описи и переводчиковъ съ еврейскаго на русскій языкъ —
-            <persName ref="Невахович">Лейбы Неваховича</persName> и <persName ref="Файбишевич">Юды Файбишевича</persName>.
+            <persName ref="Лейба_Невахович">Лейбы Неваховича</persName> и <persName ref="Файбишевич">Юды Файбишевича</persName>.
         </item>
 
         <item><label>№ 20.</label> <date when="1800-12-??">Декабрь 1800</date>. (Получено прокуроромъ

diff --git a/docs/collections/dubnov/tei/269.xml b/docs/collections/dubnov/tei/269.xml
@@ -54,7 +54,7 @@
         бумагамъ и алфавитный регистръ
         приложенъ тамъ же. А кромѣ сихъ и краткаго сочиненія обрядовъ еврейской религіи, выбраннаго изъ древнихъ
         еврейскихъ книгъ, никакихъ другихъ, а паче противныхъ еврейскому закону или благу народному нѣтъ, – въ чемъ подписуемся:
-        переводчикъ съ еврейскаго еврей <persName ref="Невахович">Лейба Неваховичъ</persName>, переводчикъ съ еврейскаго
+        переводчикъ съ еврейскаго еврей <persName ref="Лейба_Невахович">Лейба Неваховичъ</persName>, переводчикъ съ еврейскаго
         еврей <persName ref="Файбишевич">Юда Файбишовичъ</persName>.</p>
       <p>Регистръ письмамъ:</p>
       <p>1-е) Письмецо <persName ref="alter-rebbe">Боруховича</persName> къ евреямъ Давиду и Абелю о присылкѣ на письмѣ
@@ -81,7 +81,7 @@
       <p>
         <pb xml:id="p270" n="270" facs="http://facsimiles.alter-rebbe.org/facsimiles/dubnov/270.jpg"/>
         Что точно сей регистръ сходствуетъ съ писаннымъ, подписуемся: переводчикъ еврейскаго языка
-        <persName ref="Невахович">Лейба Неваховичъ</persName>,
+        <persName ref="Лейба_Невахович">Лейба Неваховичъ</persName>,
         переводчикъ еврейскаго языка <persName ref="Файбишевич">Юда Файбишовичъ</persName>.</p>
       <note type="gloss">(Далѣе идетъ русскій переводъ упомянутой въ № <ref
         target="/collections/dubnov/documents/263.2.html" role="documentViewer">14</ref> регистра молитвы,
@@ -111,7 +111,7 @@
         ברכה" דינים . – שיטה מקובצת. – ,ראשית חכמה". - .תקוני זהר". -- קצור של"ה . - שבילי אמונה,– שערי ציון, תפלות -
         ברתי ופלתי . - מדרש רבה . - שו רשב"א . . . ועוד חלקים מתנ"ך, תלמוד בבלי, דובליקאטום מהנ"ל וכו' . </p>
       <p>Что сей регистръ точно соотвѣтствуетъ прописаннымъ книгамъ, въ чемъ подписуемся. Переводчикъ еврейскаго яз.
-        <persName ref="Невахович">Лейба Неваховичъ</persName>... <persName ref="Файбишевич">Юда Файбишовичъ</persName>.</p>
+        <persName ref="Лейба_Невахович">Лейба Неваховичъ</persName>... <persName ref="Файбишевич">Юда Файбишовичъ</persName>.</p>
     </body>
   </text>
 </TEI>
diff --git a/docs/collections/rgia413/tei/011.xml b/docs/collections/rgia413/tei/011.xml
@@ -74,7 +74,7 @@
                 <l>14. молитва по еврейскіи сочиненная о благо</l>
                 <l>денствіи и спасеніи Государя._______________</l>
                 <l>Что очно сей регистръ сходствуетъ съ <unclear>подлинн</unclear>,</l>
-                <l>подписуемся. <signed>переводчикъ еврейскаго языка <persName ref="Невахович">Лейба<lb/>
+                <l>подписуемся. <signed>переводчикъ еврейскаго языка <persName ref="Лейба_Невахович">Лейба<lb/>
                     _________ неваховичъ</persName></signed></l>
                 <l><signed>Переводчикъ еврейскаго языка Юда файбишович</signed></l>
             </p>

diff --git a/docs/collections/rgia413/tei/013.xml b/docs/collections/rgia413/tei/013.xml
@@ -98,7 +98,7 @@
 				<l>праведность же оцарствуетъ царя. – Заступленіе закона</l>
 				<l>и пророковъ заступила бы ВСЕМИЛОСТИВѢЙШАГО –</l>
 				<l>ГОСУДАРЯ НАШЕГО, и да возвысилась бы слава его!!! –</l>
-				<l>Переводилъ съ еврейскаго переводчикъ <persName ref="Невахович">Лейба Неваховичъ</persName></l>
+				<l>Переводилъ съ еврейскаго переводчикъ <persName ref="Лейба_Невахович">Лейба Неваховичъ</persName></l>
 			</p>
 		</body>
 	</text>

diff --git a/docs/names.xml b/docs/names.xml
@@ -202,6 +202,11 @@
             <persName>Лейба Меіерович</persName>
           </ref>
         </l>
+        <l>
+          <ref target="/names/Лейба_Невахович.html" role="namesViewer">
+            <persName>Лейба Невахович</persName>
+          </ref>
+        </l>
         <l>
           <ref target="/names/Лейба_Юделевич.html" role="namesViewer">
             <persName>Лейба Юделевич</persName>
@@ -247,11 +252,6 @@
             <persName>Моше</persName>
           </ref>
         </l>
-        <l>
-          <ref target="/names/Невахович.html" role="namesViewer">
-            <persName>Лейба Невахович</persName>
-          </ref>
-        </l>
         <l>
           <ref target="/names/Нота_Ноткин.html" role="namesViewer">
             <persName>Ноткин, Нота Хаимович</persName>

diff --git a/docs/names/Лейба_Невахович.html b/docs/names/Лейба_Невахович.html
@@ -0,0 +1,5 @@
+---
+layout: tei
+tei: 'Лейба_Невахович.xml'
+target: namesViewer
+---
diff --git a/docs/names/Невахович.xml → docs/names/Лейба_Невахович.xml b/docs/names/Невахович.xml → docs/names/Лейба_Невахович.xml
@@ -24,17 +24,17 @@
   </teiHeader>
   <text>
     <body>
-      <person xml:id="Невахович" role="jew">
+      <person xml:id="Лейба_Невахович" role="jew">
         <persName>Лейба Невахович</persName>
         <persName>Невахович, Лев Николаевич</persName>
         <ref target="https://ru.wikipedia.org/wiki/Невахович,_Лев_Николаевич">[Википедия]</ref>
-        <ref target="http://ejwiki.org/wiki/Невахович,_Иехуда_Лейб_Бен_Hoax">[ЕжеВиКа,_Еврейская_Вики-энциклопедия]</ref>
-        <ref target="https://eleven.co.il/jewish-literature/in-russian/12937/">[Электронная_еврейская_энциклопедия]</ref>
-        <ref target="http://ju.org.ua/ru/publicism/608.html"/>
-        <ref target="http://www.jewhistory.ort.spb.ru/rus/main/s.php?id=810">[Евреи_Петербурга._Три_века_истории]</ref>
+        <ref target="http://ejwiki.org/wiki/Невахович,_Иехуда_Лейб_Бен_Hoax">[ЕжеВиКа, Еврейская Вики-энциклопедия]</ref>
+        <ref target="https://eleven.co.il/jewish-literature/in-russian/12937/">[Электронная еврейская энциклопедия]</ref>
+        <ref target="http://ju.org.ua/ru/publicism/608.html">publicism</ref>
+        <ref target="http://www.jewhistory.ort.spb.ru/rus/main/s.php?id=810">[Евреи Петербурга. Три века истории]</ref>
         <p>Переводчик с иврита на русский язык показаний участников следствия в 1800 г.</p>
         <p rendition="mentions">
-          <ref target="/names.html#Невахович" role="namesViewer">[...]</ref>
+          <ref target="/names.html#Лейба_Невахович" role="namesViewer">[...]</ref>
           <l>
             Дубнов:
             <ref target="/collections/dubnov/documents/088.html" role="documentViewer">088</ref>

diff --git a/docs/names/Невахович.html b/docs/names/Невахович.html