Fix repository's pull request count error (#7518) (#7524)

* fix pr count error

* fix tests

remove duplicated webhook trigger (#7511) (#7516)

Backport drone fix from #7480 and #7496 (#7504)

* don't make release-version deps on transalations since translations is only triggered by push on master (#7496)

* drone/docker: Use a more standard format (#7480)

* drone/docker: Use a more standard format

Based on the plugin drone structure itself : https://github.com/drone-plugins/drone-docker/blob/ebce953fc443371d79b5a019fcc9c1976f60a09a/.drone.yml#L9
Use autotag : http://plugins.drone.io/drone-plugins/drone-docker/#autotag

* use latest plugins/docker:linux-amd64

* remove useless cache_from

* Don't depends on translations step

Fixes #7474 - Handles all redirects for Web UI File CRUD (#7478) (#7507)

* Fixes #7474 - Handles all redirects for Web UI File CRUD

* Fixes lint errors

* Typo fix

* Adds unit tests for a few helper functions

* Fixes per review

* Fix for new branch creation and to unit test

* Fixes the template used for errors on delete

Backport for #7475 - settings error (#7483)

Fetch refs for successful testing for tag (#7388)

1.9.0-rc2 changelog (#7467)

* 1.9.0-rc2 changelog

* Update CHANGELOG.md

Fix regex for issues in commit messages (#7444) (#7466)

* Fix regex for issues in commit messages

Use same regex as markup for matching in commits.

Fixes #7438

* make fmt

cmd/serv: actually exit after fatal errors (#7460)

Regression in 356854fc5f8d7d1a7e4d68c9e00929e9ce8aa867, where
a log.Fatal call was removed. log.Fatal calls os.Exit(1) as intended,
but without it the fail() function returns normally and execution
continues past the fatal error, causing a panic.

This is visible as a go panic log and stack trace returned to the SSH
client, which is not only ugly, it leaks server and build system
information.

Fix by removing the stray return statement so that the fail() function
always calls os.Exit(1).

Backport: 2186a99c768cf4511d9fbed0781d94d892199873
Fixes: https://github.com/go-gitea/gitea/issues/7457
Signed-off-by: Allen Wild <allenwild93@gmail.com>

Fix an issue with some pages throwing 'not defined' js exceptions #7450 (#7453)

Fix an issue introduced by cc8e7dd355c9c625b895a1b633a3a7fa5016f718

Signed-off-by: Gary Kim <gary@garykim.dev>

fix Dropzone.js integration (#7445) (#7448)

backport of #7393 - create class for inline positioned lists (#7439)

see #7393
affects #6464

Signed-off-by: Michael Gnehr michael@gnehr.de

Diff: Fix indentation on unhighlighted code (#7435) (#7443)

There was a missing space before the `nohighlight` class which made the
previous class concatenate wrongly as `wrapnohighlight` in the template.

backport #7425 - jquery 3 (#7442)

Only show "New Pull Request" button if repo allows pulls (#7426) (#7432)

Signed-off-by: Gary Kim <gary@garykim.dev>

Fix vendor (#7394) (#7396)

only return head: null if source branch was deleted (#6705) (#7376)

* only return head: null if source branch was deleted

* add URL into GetPullRequest

* TestPullRequest_APIFormat

* log error if it is not Err(Branch)NotExist

(cherry picked from commit ff85a6331e02338100e3799b5787bf8b686fe003)

backport of #7385 - add missing template variable on organisation settings (#7386)

affects #6755

Signed-off-by: Michael Gnehr <michael@gnehr.de>

backport of #7380 (#7383)

fix post parameter - on issue list - unset assignee

Signed-off-by: Michael Gnehr <michael@gnehr.de>

Fix migration tests since #7 fixed (#7375) (#7381)

* fix migration tests since #7 fixed

* fix test time

1.9.0-RC1 changelog (#7367)

[skip ci] Updated translations via Crowdin

ui - issue view - fix icon position (#7354)

* issue view - fix icon position

- move style from template to css
- add bullets to: key, circle-slash, comment

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* add border to symbols

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* fix circle slash position

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* fix top margin

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* changed mixed space/tab indent to tabindent only

Signed-off-by: Michael Gnehr <michael@gnehr.de>

ui - cut timeline length with last element on issue view (#7355)

* cut timeline length with last element on issue view

fix #7304

- lightly enlight dark theme issue timeline color

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* remove new container

Signed-off-by: Michael Gnehr <michael@gnehr.de>

Fix typo in PR migration check (#7368)

Fix mirror sync not automatically sending webhook requests (#7366)

Make captcha and password optional for external accounts (#6606)

Detect migrating batch size (#7353)

* Make migrating batch size as configurable

* detect different table batch insert size and remove config item

* remove unrelated changes

routers: do not leak secrets via timing side channel (#7364)

* routers: do not leak secrets via timing side channel

* routers/repo: do not leak secrets via timing side channel

routers/user: ensure that decryption of cookie actually suceeds (#7363)

Previously, only the first return value of ctx.GetSuperSecureCookie
was used to check whether decryption of the auth cookie succeeded.
ctx.GetSuperSecureCookie also returns a second value, a boolean,
indicating success or not. That value should be checked first to
be on the safe side and not rely on internal logic of the encryption
and decryption blackbox.

workaround broken drone build (#7362)

* workaround broken swagger

only master brach is not working, latest release seems to work

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* make vendor

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* Don't export GO111MODULE

* set go-swagger to fixed release version

mentioned here: https://github.com/go-gitea/gitea/pull/7362#discussion_r300831537

Signed-off-by: Michael Gnehr <michael@gnehr.de>

update contributing to use recent version of drone cli (#7351)

fix #7350

fix api route (#7346)

update dependencies and various tweaks (#7344)

- ran `make npm-update`
- ran `make js`, fixed new lint issue
- ran `make css`, this added back some vendor prefixes
- added `engines` property to package.json to specify minimum required
  Node.js version
- added `private` property to package.json to prevent accidential
  publishing to npm

Display Github support button (#7343)

This need an option to be enabled on repo : https://help.github.com/en/articles/displaying-a-sponsor-button-in-your-repository

Use commit graph files for listing pages (#7314)

* Experimental support for git commit graph files and bloom filter index

Signed-off-by: Filip Navara <filip.navara@gmail.com>
* Force vendor of commitgraph

Signed-off-by: Filip Navara <filip.navara@gmail.com>
* Remove bloom filter experiment and debug prints

* Remove old code for building commit graphs

* Remove unused function

* Remove mmap usage

* gofmt

* sort vendor/modules.txt

* Add copyright header and log commit-graph error

Fix bug conflict between SyncReleasesWithTags and InsertReleases (#7337)

* fix bug conflict between SyncReleasesWithTags and InsertReleases

* fix tests

* fix fmt

make dropTableColumns drop columns on sqlite and constraints on all (#6849)

fix updated drone file (#7336)

fix pull view ui merge section (#7335)

* fix pull view ui merge section

* fix tests

* fix tests

wrap long texts on user profile info (#7333)

fix #7255

discussed in PR #7254

Signed-off-by: Michael Gnehr <michael@gnehr.de>

#6946 Run hooks on merge/edit and cope with protected branches (#6961)

* Fix #6946 by checking PullRequest ID on pushing

* Ensure we have the owner name, the pr attributes and the the issue

* Fix TestSearchRepo by waiting till indexing is done

* Update integrations/repo_search_test.go

* changes as per @mrsdizzie

* missing comma

* Spelling mistake

* Fix full pushing environment

chore: update drone format to 1.0 (#6602)

* chore: update drone format to 1.0

* Converted old drone config

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Moved translations to seperate pipeline

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Split the steps in multiple pipelines and add dependencies

Signed-off-by: konrad <konrad@kola-entertainments.de>
* format

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Fixed step depenednecies

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Fixed discord notify

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Exclude translations and docs pipelines from prs

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Fixed exclude prs

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Fixed exclude prs

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Moved releases to seperate pipeline

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Re-trigger drone

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Disable release step

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Fixed depending step

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Adopted dependencies

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Changed pipeline conditions

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Replaced pipeline conditions with triggers

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Fixed coverage step

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Import changes from #7331

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Add comment for lowest go version

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Exclude fetch-tags from prs

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Re-added comment about no tags

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Removed unneeded conditions

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Moved release version and release latest to different pipelines

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Fixed depending pipeline

Signed-off-by: konrad <konrad@kola-entertainments.de>
* Removed the conditions for all services

Signed-off-by: konrad <konrad@kola-entertainments.de>

[skip ci] Updated translations via Crowdin

1.8.3 release (#7332)

[skip ci] Updated translations via Crowdin

fix topic bar to allow prefixes (#7325)

* - do not select if escape is pressed
- allow prefixes by adding current request content to result list
- remove html-tags before insert on page

fix #7126

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* sort current query to top

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* remove already added topics from dropdown list

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* protoct against xss

thanks to @silverwind

Signed-off-by: Michael Gnehr <michael@gnehr.de>

Add commit statuses reports on pull request view (#6845)

* Add commit statuses reports on pull view

* Add some translations

* improve the UI

* fix fmt

* fix tests

* add a new test git repo to fix tests

* fix bug when headRepo or headBranch missing

* fix tests

* fix tests

* fix consistency

* fix tests

* fix tests

* change the test repo

* fix tests

* fix tests

* fix migration

* keep db size consistency

* fix translation

* change commit hash status table unique index

* remove unused table

* use char instead varchar

* make hashCommitStatusContext private

* split merge section with status check on pull view ui

* fix tests; fix arc-green theme on pull ui

update mssql drive to last working version 20180314172330-6a30f4e59a44 (#7306)

[skip ci] Updated translations via Crowdin

Fixes #7292 - API File Contents bug (#7301)

[skip ci] Updated translations via Crowdin

Fixes #7152 - Allow create/update/delete message to be empty, use default message (#7324)

* Fixes #7152 - Allow create/update/delete message to be empty, use default message

* Linting fix

* Fix to delete integration tests

Use batch insert on migrating repository to make the process faster (#7050)

* Use batch insert on migrating repository to make the process faster

* fix lint

* fix tests

* fix comments

[skip ci] Updated translations via Crowdin

add commitgraph support global default true when git version >= 2.18 (#7313)

[skip ci] Updated translations via Crowdin

Fixes #7238 - Annotated tag commit ID incorrect (#7321)

* Fixes #7238 - Annotated tag commit ID incorrect

* Fixes #7238 - Annotated tag commit ID incorrect

[skip ci] Updated translations via Crowdin

Fix 7303 - remove unnessesary buttons on archived repos (#7326)

* archived repo - remove
- open/close button on issue list
- assigne person on issue list

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* archived repo - remove
- comment field on issue view
- lock/unlock issue conversation button from sidebar on issue view

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* archived repo - add 'compare commits' button to pull request

+ remove new pull request button from compare view

as the route is still working, and there is no need to be hidden

Signed-off-by: Michael Gnehr <michael@gnehr.de>

[skip ci] Updated translations via Crowdin

Dark theme fixes (#7319)

* add dark theme for jquery xdsoft_datetimepicker

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* add border to dark theme - not checked issue checkboxes

Signed-off-by: Michael Gnehr <michael@gnehr.de>

gitea own dark codemirror theme (#7317)

* modify dark codemirror theme

fix #6573

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* add selection color

Signed-off-by: Michael Gnehr <michael@gnehr.de>

Show Pull Request button or status of latest PR in branch list (#6990)

* Show Pull Request button or status of latest PR in branch list

Signed-off-by: Mario Lubenka <mario.lubenka@googlemail.com>
* Do not show pull request button on deleted branches

Signed-off-by: Mario Lubenka <mario.lubenka@googlemail.com>
* Do not show commit divergence on deleted branches

Signed-off-by: Mario Lubenka <mario.lubenka@googlemail.com>
* Use XORMs Get instead of limit

* Links pull request ID and use smaller labels for displaying the pull request status

Signed-off-by: Mario Lubenka <mario.lubenka@googlemail.com>
* Handle error when getting latest pull request

Signed-off-by: Mario Lubenka <mario.lubenka@googlemail.com>
* Indent template

Signed-off-by: Mario Lubenka <mario.lubenka@googlemail.com>
* Check error when loading issue

Signed-off-by: Mario Lubenka <mario.lubenka@googlemail.com>

replace lesshint with stylelint (#7305)

New CSS linter which is much more powerfull than the previous one.
Configuration is default but I had to remove a few rules that were
throwing too many or weird errors.

More importantly, the linter will exit with code 1 on errors so now our
build will fail if the CSS linter fails which should eliminate linter
errors being introduced without notice.

fix extra newlines when copying from diff in Firefox (#7288)

* fix extra newlines when copying from diff

See https://bugzilla.mozilla.org/show_bug.cgi?id=1273836

Basically, the <pre><code> seems to add a forced newline that is not
possible to get rid of via CSS, so I replaced it with just a <code>.

Secondly, .lines-type-marker also forced a newline in the copied text,
but that was possible to get rid of via user-select.

Safari still has a extraneous newline in the copied text of unknown
origin, but this should not block stop this PR.

* simplify .line-type-marker

* fix selector

* remove erronous ^^^

* Fix empty split diff

* Fix arc-theme-green

* fix add comment

* ensure line-num is copied too

* Update templates/repo/diff/box.tmpl

Co-Authored-By: zeripath <art27@cantab.net>
* attempt to fix safari via removing <code>

* remove useless whitespace at the end of 'class'

* remove inter-tag whitespace for code <td>s

* more inter-tag removal

* final inter-tag removal

* attempt to fix empty line copy

* move and comment getLineContent

* fix golint

* make background grey for missing added code

Monitor all git commands; move blame to git package and replace git as a variable (#6864)

* monitor all git commands; move blame to git package and replace git as a variable

* use git command but not other commands

* fix build

* move exec.Command to git.NewCommand

* fix fmt

* remove unrelated changes

* remove unrelated changes

* refactor IsEmpty and add tests

* fix tests

* fix tests

* fix tests

* fix tests

* remove gitLogger

* fix fmt

* fix isEmpty

* fix lint

* fix tests

Shadow the password on cache and session config on admin panel (#7300)

* shadow the password on cache and session config on admin panel

* add shadow password of mysql/postgres/couchbase

* fix log import

fix API link header (#7298)

Make diff line-marker non-selectable (#7279)

* Make diff line-marker non-selectable
* Move to use data-* as per @mrsdizzie
* fix missing line nums
* Add a minimum-width to force right-align of the line num
* Move line-type-marker into separate column

[docker] Add LFS_START_SERVER option to control git-lfs support (#7281)

Fix error log when loading issues caused by a xorm bug (#7271)

* fix error log when loading issues caused by a xorm bug

* upgrade packages

* fix fmt

* fix Consistency

* fix tests

Fix #732: Add LFS objects to base repository on merging  (#7082)

On merge we walk the merge history and ensure that all lfs objects pointed to in
the history are added to the base repository. This switches from relying on having git-lfs installed on the server, (and in fact .gitattributes being correctly installed.)

Dark theme markdown fixes (#7260)

* fix colors in dark theme - tables inside rendered markdown

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* fix colors in dark theme - simpleMDE rendered preview

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* Update public/less/themes/arc-green.less

remove scroll bar color, to add this globally on additional PR

Co-Authored-By: Lauris BH <lauris@nix.lv>
Signed-off-by: Michael Gnehr <michael@gnehr.de>
* add missing class name

Signed-off-by: Michael Gnehr <michael@gnehr.de>

Fix Submodule dection in subdir (#7275)

Include the directory in name when looking for matching Submodule from
.gitmodules file

Fixes #7262

Alpine 3.10 (#7256)

add .fa margin like .octicon (#7258)

Only check and config git on web subcommand but not others (#7236)

* only check and config git on web subcommand but not others

* add Init in git tests

add 'npm' and 'npm-update' make targets and lockfile (#7246)

* add 'npm' and 'npm-update' make targets and lockfile

- `make npm` installs and updates node_modules, triggered automatically
   on `make css` and `make js` as it completes reasonably fast and
   ensures consistent modules.
- `make npm-update` updates all dependencies to their latest version,
   regenerates `node_modules` from scratch and updates
   `package-lock.json`. It uses npm modules `updates` written by yours
   truly to find the latest version of each dependency.

* add suggested make dependencies

* remove package-lock.json during npm-update

* regenerate package-lock.json

update go-git to v4.12.0 - fixes #7248 (#7249)

fix hljs unintenionally highlighting commit links (#7244)

* fix hljs unintenionally highlighting commit links

* fix unit tests

add self to MAINTAINERS (#7245)

Fix migration panic when Head.User is not exist (#7226)

* fix migration panic when Head.User is not exist

* fix test

Update css (#7240)

Remove -o-tab-size selectors from tab-sze-1 css

Fix relref link in docs (#7233)

Add docs for `INTERNAL_TOKEN_URI` (#7234)

Changelog 1.8.3 (#7230) (#7231)

* Changelog 1.8.3

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Suggestion

Co-Authored-By: zeripath <art27@cantab.net>

Add CLI commands to manage LDAP authentication source (#6681)

* add CLI commands to manage LDAP authentication source

* delete Gogs copyright

* remove unused return value of func parseLoginSource

* fix comment

Co-Authored-By: ngourdon <31291059+ngourdon@users.noreply.github.com>
* remove config flag already present in global flags

* remove config flag from ldap commands in docs

* remove config flag handling

[skip ci] Updated translations via Crowdin

Use certmanager provided TLSConfig for LetsEncrypt (#7229)

show lfs config on admin panel (#7220)

Add missing description of label on API (#7159)

* add missing description of label on API

* fix comment head

* fix swagger

Use go method to calculate ssh key fingerprint (#7128)

* Use go method to calculate key fingerprint

* add gitea copyright

* use native go method only for built-in server

* refactor and add tests

* add gitea copyright

Add LastLogin to the User API (#7196)

setting: don't require same running user for internal SSH (like win platform) (#7215)

Only warn on errors in deleting LFS orphaned files during repo deletion (#7213)

fix duplicated file on pull request conflicted files (#7211)

Allow colon between fixing word and issue (#7207)

* Allow colon between fixing word and issue

* update test

UI: Fix overflow issues in repo (#7190)

- Fix layout overflow in repo file list.
- Fix invisible status icon in file view and commit list. In file view,
  the icon was moved to the left because I could not figure out a proper
  fix because of HTML tables.
- Added title attribute to commit messages.
- Fixed two CSS linter warnings in existing CSS.
- Fixed CI variable check in 'make css'.

Fixes: https://github.com/go-gitea/gitea/issues/7180

fix drone build bug (#7192)