Project.pm: allow _new for all does_exist projects
authorKyle J. McKay <mackyle@gmail.com>
Wed, 7 Aug 2013 17:30:25 +0000 (7 10:30 -0700)
committerKyle J. McKay <mackyle@gmail.com>
Wed, 7 Aug 2013 17:30:25 +0000 (7 10:30 -0700)
The valid_name check is more restrictive than the does_exist check which
prevents loading some projects that actually exist (e.g. a fork that has
been kept but has had its forkee removed).

If the project actually does_exist go ahead and allow the _new to succeed.

Girocco/Project.pm

index ce926f1..1208c59 100644 (file)
@@ -306,7 +306,7 @@ sub _hooks_install {
 sub _new {
        my $class = shift;
        my ($name, $base_path, $path) = @_;
-       valid_name($name) or die "refusing to create project with invalid name ($name)!";
+       does_exist($name,1) || valid_name($name) or die "refusing to create project with invalid name ($name)!";
        $path ||= "$base_path/$name.git";
        my $proj = { name => $name, base_path => $base_path, path => $path };