diff options
| author | Christian Weiske <cweiske@cweiske.de> | 2012-10-24 15:16:24 +0200 |
|---|---|---|
| committer | Christian Weiske <cweiske@cweiske.de> | 2012-10-24 15:16:24 +0200 |
| commit | 860aaa0c3f990fd888bb779f60d24b39d8adaae9 (patch) | |
| tree | d4084de19e87040c8bec97c6cd8867453ccfda78 | |
| parent | f1d6f647c38d0ebf78f812bf551e0ab2b4f48a70 (diff) | |
| download | phorkie-860aaa0c3f990fd888bb779f60d24b39d8adaae9.tar.gz phorkie-860aaa0c3f990fd888bb779f60d24b39d8adaae9.zip | |
show proper error message on remote forking
| -rw-r--r-- | src/phorkie/ForkRemote.php | 12 | ||||
| -rw-r--r-- | www/fork-remote.php | 2 |
2 files changed, 12 insertions, 2 deletions
diff --git a/src/phorkie/ForkRemote.php b/src/phorkie/ForkRemote.php index 78c3caa..7f6129d 100644 --- a/src/phorkie/ForkRemote.php +++ b/src/phorkie/ForkRemote.php @@ -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; } /** diff --git a/www/fork-remote.php b/www/fork-remote.php index 0567748..5d4a4ee 100644 --- a/www/fork-remote.php +++ b/www/fork-remote.php @@ -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']); |
