Extract Clip_Doctrine_Pubdata->validateUrltitle to be used where need…

…ed. Refs #9 and #56

src/modules/Clip/lib/Clip/Doctrine/Pubdata.php

@@ -453,6 +453,28 @@ public function notifyHooks($hooktype)
         ServiceUtil::getManager()->getService('zikula.hookmanager')->notify($hook);
     }
 
+    /**
+     * urltitle corrector.
+     *
+     * @return void
+     */
+    public function validateUrltitle()
+    {
+        if (!$this->core_urltitle) {
+            return;
+        }
+
+        // validate the unique urltitle
+        $pid = $this->getTable()->selectFieldBy('core_pid', $this->core_urltitle, 'core_urltitle');
+
+        while ($pid && $pid != $this->core_pid) {
+            // TODO better to throw a validation exception
+            ++$this->core_urltitle;
+
+            $pid = $this->getTable()->selectFieldBy('core_pid', $this->core_urltitle, 'core_urltitle');
+        }
+    }
+
     /**
      * preInsert hook.
      *
@@ -501,15 +523,7 @@ public function preSave($event)
             $pub['core_urltitle'] = substr(DataUtil::formatPermalink($urltitle), 0, 255);
         }
 
-        // validate the unique urltitle
-        $pid = $this->getTable()->selectFieldBy('core_pid', $pub['core_urltitle'], 'core_urltitle');
-
-        while ($pid && $pid != $pub['core_pid']) {
-            // TODO better to throw a validation exception
-            ++$pub->core_urltitle;
-
-            $pid = $this->getTable()->selectFieldBy('core_pid', $pub['core_urltitle'], 'core_urltitle');
-        }
+        $pub->validateUrltitle();
 
         // invoke the preSave hook on pubfields
         if (isset($pub['core_tid'])) {