| ... | ... |
@@ -26,7 +26,7 @@ |
| 26 | 26 |
my $target_user_id; |
| 27 | 27 |
my $target_branch; |
| 28 | 28 |
my $target_project; |
| 29 |
- if ($user_id_and_target_branch =~ /^([^:]+):(.+)/) {
|
|
| 29 |
+ if (defined $user_id_and_target_branch && $user_id_and_target_branch =~ /^([^:]+):(.+)/) {
|
|
| 30 | 30 |
$target_user_id = $1; |
| 31 | 31 |
$target_branch = $2; |
| 32 | 32 |
$target_project = $self->app->manager->child_project($base_user_id, $base_project_id, $target_user_id); |
| ... | ... |
@@ -59,13 +59,15 @@ |
| 59 | 59 |
%= include '/include/header'; |
| 60 | 60 |
|
| 61 | 61 |
<div class="container"> |
| 62 |
- <div class="pulls-button-container"> |
|
| 63 |
- % if ($original_project) {
|
|
| 64 |
- <a href="<%= url_for("/$original_project->{'user.id'}/$original_project->{id}/compare/$original_project->{default_branch}...$user_id:$default_branch") %>" class="btn btn-success">New pull request</a>
|
|
| 65 |
- % } else {
|
|
| 66 |
- <a href="<%= url_for("/$user_id/$project_id/compare") %>" class="btn btn-success">New pull request</a>
|
|
| 67 |
- % } |
|
| 68 |
- </div> |
|
| 62 |
+ % if ($api->logined) {
|
|
| 63 |
+ <div class="pulls-button-container"> |
|
| 64 |
+ % if ($original_project) {
|
|
| 65 |
+ <a href="<%= url_for("/$original_project->{'user.id'}/$original_project->{id}/compare/$original_project->{default_branch}...$user_id:$default_branch") %>" class="btn btn-success">New pull request</a>
|
|
| 66 |
+ % } else {
|
|
| 67 |
+ <a href="<%= url_for("/$user_id/$project_id/compare") %>" class="btn btn-success">New pull request</a>
|
|
| 68 |
+ % } |
|
| 69 |
+ </div> |
|
| 70 |
+ % } |
|
| 69 | 71 |
<div class="pulls"> |
| 70 | 72 |
<div class="pulls-header"> |
| 71 | 73 |
% if ($open) {
|