show proper error message on remote forking
authorChristian Weiske <cweiske@cweiske.de>
Wed, 24 Oct 2012 13:16:24 +0000 (15:16 +0200)
committerChristian Weiske <cweiske@cweiske.de>
Wed, 24 Oct 2012 13:16:24 +0000 (15:16 +0200)
src/phorkie/ForkRemote.php
www/fork-remote.php

index 78c3caa..7f6129d 100644 (file)
@@ -3,6 +3,11 @@ namespace phorkie;
 
 class ForkRemote
 {
+    /**
+     * Contains error message when parse() failed
+     */
+    public $error;
+
     protected $url;
 
     /**
@@ -90,7 +95,12 @@ class ForkRemote
             }
         }
 
-        return $count > 0;
+        if ($count > 0) {
+            return true;
+        }
+
+        $this->error = 'No git:// clone URL found';
+        return false;
     }
 
     /**
index 0567748..5d4a4ee 100644 (file)
@@ -12,7 +12,7 @@ if (isset($_POST['remote_url'])) {
     $fr = new ForkRemote($_POST['remote_url']);
     if (false === $fr->parse()) {
         //no url found
-        $error = 'No git:// clone URL found';
+        $error = $fr->error;
     } else if (false !== ($gitUrl = $fr->getUniqueGitUrl())) {
         if (isset($_POST['orig_url'])) {
             $fr->setUrl($_POST['orig_url']);