fix canCreateBranch/disableCreateBranch

go-gitea · Mar 10, 2023 · b05907e · b05907e
1 parent 703dfd3
commit b05907e
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 8 deletions.
diff --git a/templates/repo/branch_dropdown.tmpl b/templates/repo/branch_dropdown.tmpl
@@ -16,7 +16,7 @@
  'showBranchesInDropdown': {{$showBranchesInDropdown}},
  'searchFieldPlaceholder': '{{if $.noTag}}{{.root.locale.Tr "repo.pulls.filter_branch"}}{{else if $showBranchesInDropdown}}{{.root.locale.Tr "repo.filter_branch_and_tag"}}{{else}}{{.root.locale.Tr "repo.find_tag"}}{{end}}...',
  'branchForm': {{$.branchForm}},
- 'canCreateBranch': {{if .canCreateBranch}}{{.canCreateBranch}}{{else}}{{.root.CanCreateBranch}}{{end}},
+ 'disableCreateBranch': {{if .disableCreateBranch}}{{.disableCreateBranch}}{{else}}{{not .root.CanCreateBranch}}{{end}},
  'setAction': {{.setAction}},
  'submitForm': {{.submitForm}},
  'viewType': {{$type}},

diff --git a/templates/repo/commit_page.tmpl b/templates/repo/commit_page.tmpl
@@ -52,24 +52,24 @@
  data-modal-cherry-pick-type="revert"
  data-modal-cherry-pick-header="{{$.locale.Tr "repo.commit.revert-header" (ShortSha .CommitID)}}"
  data-modal-cherry-pick-content="{{$.locale.Tr "repo.commit.revert-content"}}"
- data-modal-cherry-pick-submit="{{.locale.Tr "repo.commit.revert"}}">{{.locale.Tr "repo.commit.revert"}}</a></div>
+ data-modal-cherry-pick-submit="{{.locale.Tr "repo.commit.revert"}}">{{.locale.Tr "repo.commit.revert"}}</div>
  <div class="item cherry-pick-button show-modal"
  data-modal="#cherry-pick-modal"
  data-modal-cherry-pick-type="cherry-pick"
  data-modal-cherry-pick-header="{{$.locale.Tr "repo.commit.cherry-pick-header" (ShortSha .CommitID)}}"
  data-modal-cherry-pick-content="{{$.locale.Tr "repo.commit.cherry-pick-content"}}"
- data-modal-cherry-pick-submit="{{.locale.Tr "repo.commit.cherry-pick"}}">{{.locale.Tr "repo.commit.cherry-pick"}}</a></div>
+ data-modal-cherry-pick-submit="{{.locale.Tr "repo.commit.cherry-pick"}}">{{.locale.Tr "repo.commit.cherry-pick"}}</div>
  <div class="ui basic modal" id="cherry-pick-modal">
  <div class="ui icon header">
  <span id="cherry-pick-header"></span>
  </div>
  <div class="content center">
  <p id="cherry-pick-content" class="branch-dropdown"></p>
  {{template "repo/branch_dropdown" dict "root" .
- "noTag" "true" "canCreateBranch" "false"
+ "noTag" true "disableCreateBranch" true
  "branchForm" "branch-dropdown-form"
  "branchURLPrefix" (printf "%s/_cherrypick/%s/" $.RepoLink .CommitID) "branchURLSuffix" ""
- "setAction" "true" "submitForm" "true"}}
+ "setAction" true "submitForm" true}}
  <form method="GET" action="{{$.RepoLink}}/_cherrypick/{{.CommitID}}/{{if $.BranchName}}{{PathEscapeSegments $.BranchName}}{{else}}{{PathEscapeSegments $.Repository.DefaultBranch}}{{end}}" id="branch-dropdown-form">
  <input type="hidden" name="ref" value="{{if $.BranchName}}{{$.BranchName}}{{else}}{{$.Repository.DefaultBranch}}{{end}}">
  <input type="hidden" name="refType" value="branch">

diff --git a/web_src/js/components/RepoBranchTagDropdown.js b/web_src/js/components/RepoBranchTagDropdown.js
@@ -21,12 +21,14 @@ export function initRepoBranchTagDropdown(selector) {
  ...window.config.pageData.branchDropdownDataList[dropdownIndex],
  };
 
- if (data.showBranchesInDropdown) {
+ // the "data.defaultBranch" is ambiguous, it could be "branch name" or "tag name"
+
+ if (data.showBranchesInDropdown && data.branches) {
  for (const branch of data.branches) {
  data.items.push({name: branch, url: branch, branch: true, tag: false, selected: branch === data.defaultBranch});
  }
  }
- if (!data.noTag) {
+ if (!data.noTag && data.tags) {
  for (const tag of data.tags) {
  if (data.release) {
  data.items.push({name: tag, url: tag, branch: false, tag: true, selected: tag === data.release.tagName});
@@ -56,7 +58,7 @@ export function initRepoBranchTagDropdown(selector) {
  return this.filteredItems.length === 0 && !this.showCreateNewBranch;
  },
  showCreateNewBranch() {
- if (!this.canCreateBranch || !this.searchTerm) {
+ if (this.disableCreateBranch || !this.searchTerm) {
  return false;
  }