implement create() template

openscholar · Feb 28, 2019 · 0ff27d2 · 0ff27d2
1 parent 9a7cb58
commit 0ff27d2
Show file tree

Hide file tree

Showing 3 changed files with 62 additions and 24 deletions.
diff --git a/src/Repec.php b/src/Repec.php
@@ -662,32 +662,14 @@ private function createPaperTemplate(ContentEntityInterface $entity) {
     /** @var array $template */
     $template = $this->getEntityTemplate($entity);
 
-    $serieDirectoryConfig = $this->getEntityBundleSettings('serie_directory', $entity->getEntityTypeId(), $entity->bundle());
-    $directory = $this->getArchiveDirectory() . $serieDirectoryConfig . '/';
-
-    if (!empty($directory) &&
-      file_prepare_directory($directory, FILE_CREATE_DIRECTORY)) {
-
-      $fileName = $serieDirectoryConfig . '_' . $entity->getEntityTypeId() . '_' . $entity->id() . '.rdf';
-
-      $content = '';
-      foreach ($template as $item) {
-        if (!empty($item['value'])) {
-          $content .= $item['attribute'] . ': ' . $item['value'] . "\n";
-        }
-      }
-
-      if (!file_put_contents($directory . '/' . $fileName, $content)) {
-        $this->messenger->addError(t('File @file_name could not be created', [
-          '@file_name' => $fileName,
-        ]));
-      }
+    /** @var \Drupal\repec\Series\Base $template_class */
+    $template_class = $this->templateFactory->create($this->getEntityBundleSettings('serie_type', $entity->getEntityTypeId(), $entity->bundle()), $entity);
 
+    try {
+      $template_class->create($template);
     }
-    else {
-      $this->messenger->addError(t('Directory @path could not be created.', [
-        '@path' => $directory,
-      ]));
+    catch (\Exception $e) {
+      $this->messenger->addError($e->getMessage());
     }
   }
 

diff --git a/src/Series/Base.php b/src/Series/Base.php
@@ -70,4 +70,50 @@ public function __construct(ImmutableConfig $settings, EntityTypeManagerInterfac
    */
   abstract public function getDefault() : array;
 
+  /**
+   * Creates the template.
+   *
+   * @param array $template
+   *   The template structure.
+   *
+   * @throws \Drupal\repec\Series\CreateException
+   */
+  public function create(array $template) {
+    /** @var array $bundle_settings */
+    // The following is already done in
+    // \Drupal\repec\Repec::getEntityBundleSettings
+    // TODO: Move this to a parent service or something.
+    $bundle_settings = unserialize($this->settings->get("repec_bundle.{$this->entity->getEntityTypeId()}.{$this->entity->bundle()}"));
+
+    /** @var string $serie_directory_config */
+    $serie_directory_config = $bundle_settings['serie_directory'];
+
+    // The following is already done in \Drupal\repec\Repec::getArchiveDirectory
+    // TODO: Move this to a parent service or something.
+    $archive_directory = "public://{$this->settings->get('base_path')}/{$this->settings->get('archive_code')}/";
+
+    $directory = "{$archive_directory}{$serie_directory_config}/";
+
+    if (!file_prepare_directory($directory, FILE_CREATE_DIRECTORY)) {
+      throw new CreateException($this->t('Directory @path could not be created.', [
+        '@path' => $directory,
+      ]));
+    }
+
+    $file_name = "{$serie_directory_config}_{$this->entity->getEntityTypeId()}_{$this->entity->id()}.rdf";
+
+    $content = '';
+    foreach ($template as $item) {
+      if (!empty($item['value'])) {
+        $content .= $item['attribute'] . ': ' . $item['value'] . "\n";
+      }
+    }
+
+    if (!file_put_contents("$directory/$file_name", $content)) {
+      throw new CreateException($this->t('File @file_name could not be created.', [
+        '@file_name' => $file_name,
+      ]));
+    }
+  }
+
 }
diff --git a/src/Series/CreateException.php b/src/Series/CreateException.php
@@ -0,0 +1,10 @@
+<?php
+
+namespace Drupal\repec\Series;
+
+/**
+ * Class CreateException.
+ *
+ * Custom exception handler for template creation.
+ */
+class CreateException extends \Exception {}