... | ... |
@@ -363,10 +363,8 @@ sub _create_project { |
363 | 363 |
my ($self, $user, $project, $params) = @_; |
364 | 364 |
$params ||= {}; |
365 | 365 |
|
366 |
- # DBI |
|
367 |
- my $dbi = $self->app->dbi; |
|
368 |
- |
|
369 | 366 |
# Create project |
367 |
+ my $dbi = $self->app->dbi; |
|
370 | 368 |
$dbi->connector->txn(sub { |
371 | 369 |
unless (defined $params->{original_pid}) { |
372 | 370 |
my $number = $dbi->model('number')->select('value', where => {key => 'original_pid'})->value; |
... | ... |
@@ -381,10 +379,8 @@ sub _create_project { |
381 | 379 |
sub _create_rep { |
382 | 380 |
my ($self, $user, $project, $opts) = @_; |
383 | 381 |
|
384 |
- # Git |
|
385 |
- my $git = $self->app->git; |
|
386 |
- |
|
387 | 382 |
# Create repository directory |
383 |
+ my $git = $self->app->git; |
|
388 | 384 |
my $rep = $git->rep($user, $project); |
389 | 385 |
mkdir $rep |
390 | 386 |
or croak "Can't create directory $rep: $!"; |
... | ... |
@@ -524,6 +520,7 @@ sub _delete_user_dir { |
524 | 520 |
sub _delete_project { |
525 | 521 |
my ($self, $user, $project) = @_; |
526 | 522 |
|
523 |
+ # Delete project |
|
527 | 524 |
my $dbi = $self->app->dbi; |
528 | 525 |
$dbi->model('project')->delete(id => [$user, $project]); |
529 | 526 |
} |
... | ... |
@@ -531,6 +528,7 @@ sub _delete_project { |
531 | 528 |
sub _delete_rep { |
532 | 529 |
my ($self, $user, $project) = @_; |
533 | 530 |
|
531 |
+ # Delete repository |
|
534 | 532 |
my $rep_home = $self->app->git->rep_home; |
535 | 533 |
croak "Can't remove repository. repositry home is empty" |
536 | 534 |
if !defined $rep_home || $rep_home eq ''; |
... | ... |
@@ -542,7 +540,8 @@ sub _delete_rep { |
542 | 540 |
|
543 | 541 |
sub exists_project { |
544 | 542 |
my ($self, $user, $project) = @_; |
545 |
- |
|
543 |
+ |
|
544 |
+ # Exists project |
|
546 | 545 |
my $dbi = $self->app->dbi; |
547 | 546 |
my $row = $dbi->model('project')->select(id => [$user, $project])->one; |
548 | 547 |
|
... | ... |
@@ -552,6 +551,7 @@ sub exists_project { |
552 | 551 |
sub _exists_rep { |
553 | 552 |
my ($self, $user, $project) = @_; |
554 | 553 |
|
554 |
+ # Exists repository |
|
555 | 555 |
my $rep = $self->app->git->rep($user, $project); |
556 | 556 |
|
557 | 557 |
return -e $rep; |
... | ... |
@@ -560,7 +560,7 @@ sub _exists_rep { |
560 | 560 |
sub _fork_rep { |
561 | 561 |
my ($self, $user, $project, $to_user, $to_project) = @_; |
562 | 562 |
|
563 |
- # Git clone |
|
563 |
+ # Fork repository |
|
564 | 564 |
my $git = $self->app->git; |
565 | 565 |
my $rep = $git->rep($user, $project); |
566 | 566 |
my $to_rep = $git->rep($to_user, $to_project); |
... | ... |
@@ -579,12 +579,12 @@ sub _fork_rep { |
579 | 579 |
sub _rename_project { |
580 | 580 |
my ($self, $user, $project, $renamed_project) = @_; |
581 | 581 |
|
582 |
- my $dbi = $self->app->dbi; |
|
583 |
- |
|
582 |
+ # Check arguments |
|
584 | 583 |
croak "Invalid parameters" |
585 | 584 |
unless defined $user && defined $project && defined $renamed_project; |
586 | 585 |
|
587 | 586 |
# Rename project |
587 |
+ my $dbi = $self->app->dbi; |
|
588 | 588 |
$dbi->model('project')->update( |
589 | 589 |
{name => $renamed_project}, |
590 | 590 |
id => [$user, $project] |
... | ... |
@@ -600,11 +600,13 @@ sub _rename_project { |
600 | 600 |
sub _rename_rep { |
601 | 601 |
my ($self, $user, $project, $renamed_project) = @_; |
602 | 602 |
|
603 |
+ # Check arguments |
|
603 | 604 |
croak "Invalid user name or project" |
604 | 605 |
unless defined $user && defined $project && defined $renamed_project; |
606 |
+ |
|
607 |
+ # Rename repository |
|
605 | 608 |
my $rep = $self->app->git->rep($user, $project); |
606 | 609 |
my $renamed_rep = $self->app->git->rep($user, $renamed_project); |
607 |
- |
|
608 | 610 |
move($rep, $renamed_rep) |
609 | 611 |
or croak "Can't move $rep to $renamed_rep: $!"; |
610 | 612 |
} |