Avoid re-issuing redundant cross-references. (#8734)

* Avoid re-issuing redundant cross-references.

* Remove unused func; fix lint

* Simplify code as suggested by @laftriks

* Update test

fix: add search to reserved usernames (#9063)

* fix: add search to reserved usernames

* Update integrations/user_test.go

Co-Authored-By: 6543 <24977596+6543@users.noreply.github.com>

[skip ci] Updated translations via Crowdin

Allow authors to use act keywords in PR content (#9059)

Close/reopen issues by keywords in titles and comments (#8866)

* Add close/reopen from comment functionality

* Fix comment

* Rewrite closing/reopening template

* Check xref permissions, move action to services/pull

* Fix RefIsPull field

* Add xref tests

* Fix xref unique filter

* Only highlight keywords for actionable xrefs

* Fix xref neuter filter

* Fix check return status

* Restart CI

Rewrite delivery of issue and comment mails (#9009)

* Mail issue subscribers, rework the function

* Simplify a little more

* Fix unused variable

* Refactor mail delivery to avoid heavy load on server

* Avoid splitting into too many goroutines

* Fix comments and optimize GetMaileableUsersByIDs()

* Fix return on errors

Move modules/gzip to gitea.com/macaron/gzip (#9058)

* Move modules/gzip to gitea.com/macaron/gzip

* Fix vendor

[skip ci] Updated translations via Crowdin

Fix double scroll in branch dropdown (#9048)

* Fix double scroll in branch dropdown

* Revert of 425ae38aa9ea3e9a25dee235aa9176762fe4bc5b

* Changes in .less, generated new .css

* Revert unnecessary commented line

enable lazy-loading of gitgraph.js (#9036)

- moved gitgraph.js to web_src and made it importable and es6-compatible
- created new webpack chunk for gitgraph
- enabled CSS loader in webpack
- enabled async/await syntax via regenerator-runtime
- added script to ensure webpack chunks are loaded correctly
- disable terser's comment extraction to prevent .LICENCE files

gitgraph.js has many issues:

1. it is incompatible with ES6 because of strict-mode violations
1. it does not export anything
1. it's css has weird styles like for `body`
1. it is not available on npm

I fixed points 1-3 in our version so it's now loadable in webpack. We should eventually consider alternatives.

[skip ci] Updated translations via Crowdin

wiki - add 'write' 'preview' buttons to wiki edit like in issues (#7241)

* Add add 'write' 'preview' buttons to wiki edit like in issues

affects #6975

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* update dark theme

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* fix css lint warnings - missing spaces

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* hide preview button on no fullscreen toolbar

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

[skip ci] Updated translations via Crowdin

Add first issue comment hashtag (#9052)

[skip ci] Updated translations via Crowdin

Handle setting default branch via API (#9030)

Update branch API endpoint to show effective branch protection. (#9031)

* Add API endpoint for displaying effective branch protection.

* Add status checks.

integrations: fix dropped errors in TestCreateOrUpdateRepoFileForUpdateWithFileMove() (#9040)

integrations: fix dropped error in lfsCommitAndPushTest()

Only return JSON for add/remove all (#9028)

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

Check compiled files during build (#9042)

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

Button tooltip formatting under Branches (#9034)

* Fixes button tooltip formatting under Branches

* Formatting on download button for default branch

Add retry for migration http/https requests (#9019)

* Add retry for migration http/https requests

* give the more suitable name for retry configuraion items

* fix docs and lint

* Only use retryDownloader when setting > 1

wiki - editor - enable side-by-side button (#7242)

* wiki - enable side-by-side button in editor

and add some delay so side-by-side live preview is updated
* every 10th keypress
* if keypress < 10 -> apter no input for 1 sec

affects #5436

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* decrease timeinterval user need to stop before rendering is triggered

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* removed not needed code with simpleMDE placeholder

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* run highlight.js on markdown preview

Signed-off-by: Michael Gnehr <michael@gnehr.de>
* fix white border around side-by-side preview

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

Move git graph from models to modules/graph (#9027)

Update display of reactions to issues and comments (#9038)

Modify display of reactions to issues to remove spaces and make them
feel more apart of the reactions bar rather than separate items within.

extract some inline functions related with create comment (#8931)

Fix permission checks for close/reopen from commit (#8875)

* Fix checks for close/reopen from commit

* Fix permission order

Ensure Written is set in GZIP ProxyResponseWriter (#9018)

Fix #9001

The GZIP ProxyReponseWriter doesn't currently respond correctly
to requests about its Written status - leading to #9001.

This PR properly reimplements these methods.

Add review comments to mail notifications (#8996)

Fix broken link to branch from issue list (#9003)

* Fix broken link to branch from issue list

* Update templates/repo/issue/list.tmpl

Co-Authored-By: mrsdizzie <info@mrsdizzie.com>
* apply @mrsdizzie's review

fix wrong system notice when repository is empty (#9010)

Move transfer repository and rename repository on a service package and start action notification (#8573)

* move transfer repository and rename repository on a service package and start action notification

* remove unused codes

* fix lint

* fix bugs

* fix test

* fix test

* fix test

* fix lint

* update go mod and sum

Githook highlighter (#8932)

* Add code highlighter to GitHook edit

Signed-off-by: Konctantin <gawrilyako@gmail.com>
* Fix indentation

Signed-off-by: Konctantin <gawrilyako@gmail.com>
* Resolve merge conflict

Signed-off-by: gawrilyako@gmail.com <Konctantin>
* Resolve merge conflict

Signed-off-by: gawrilyako@gmail.com <Konctantin>
* Fix pre-build drone event

Signed-off-by: Konctantin <gawrilyako@gmail.com>

services/mirror: fix dropped test errors (#9007)

Expand/Collapse Files and Blob Excerpt while Reviewing/Comparing code (#8924)

* update #8659 fold/unfold code diffs

* add fold button style

* update #8659 implement expand up/down codes (blob excerpt)

* fix golint errors

* fix expand direction

* remove debug message

* update css style for blob exceprt

* fix typo in comment

* update style sheet with less

* update expect diff (add SectionInfo)

* update #8942 accept suggested change (fix typo)

* close reader and check file type before get tail section

* adjust button position and check file type before insert fold button

* move index js to web_src

* merge index.js with master

* generate index.js

* update js coding style

Pack web_src/js/draw.js to public/js/index.js (#8975)

* pack web_src/js/draw.js to public/js/index.js

* fix eslint

Update dashboard context for PR reviews (#8995)

* Update dashboard context for PR reviews

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Update options/locale/locale_en-US.ini

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>
* Only append head action if it has content or is approval/rejection

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Update options/locale/locale_en-US.ini

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>

Enforce Gitea environment for pushes (#8982)

* Enforce Gitea environment for pushes

* Update custom/conf/app.ini.sample

Co-Authored-By: Antoine GIRARD <sapk@users.noreply.github.com>

modernize js and use babel (#8973)

* modernize js and use babel

- add babel toolchain to transform modern JS to ES5
- extend eslint config for modern rules
- fixes linting issues via `eslint --fix` and manual fixes

* run 'make css' to satisfy CI

* code style tweaks and set js indendation to 2 in .editorconfig

* regenerate js

New review approvals shouldn't require a message (#8991)

* New approvals don't require content

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Fix edge-bug

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

Shadow password correctly for session config (#8984)

Fix #8718

This PR shadows passwords in session config correctly by detecting
the VirtualProvider, unmarshalling the original config and then
shadowing config within that.

Drop Admin attribute based on LDAP when login (continue #1743) (#8849)

* Update User information in Gitea based on LDAP when login

* Update Admin Flag only if exist in settings

* Fix affectation

* Update models/login_source.go

Co-Authored-By: JustKiddingCode <JustKiddingCode@users.noreply.github.com>
* Better ident

* Apply suggestions from code review

Update user information

Co-Authored-By: 6543 <24977596+6543@users.noreply.github.com>
* Make fmt

* add err handling

* if user exist but login is Prohibit return return nil, and Prohibit err

* keep login speed

* User sync is implemented at #1478 - so only make sure that admin acces is drpoed if changed

* handle error and still use async task

* no async

* only update admin if Sync is enabled

* update two comments

* add lafriks suggestions

Co-Authored-By: Lauris BH <lauris@nix.lv>
* if adminFilter is set - use it

Co-Authored-By: Lauris BH <lauris@nix.lv>
* Update models/login_source.go

well - I should look more detaild at suggestions :D

Co-Authored-By: Lauris BH <lauris@nix.lv>
* make it work again

* set is_admin value to user

* look nicer

fix bug when pack js (#8992)

Add ALLOW_ONLY_EXTERNAL_REGISTRATION to config cheat sheet (#8986)

ALLOW_ONLY_EXTERNAL_REGISTRATION was missed off the cheat sheet. This PR adds it.

Fix #8180

Changelog for 1.10.0 (#8978) (#8987)

* Changelog for 1.10.0 (#8978)

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

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

[Feature] Add Gitea Emoji (#8950)

* add gitea emoji

* alphabetic order

* Update emojify.js to 1.1.0

* mv to custom

* add gitea emoji

* minify

[skip ci] Updated translations via Crowdin

fix(docs): Fixed creating Windows service commands (#8979)

escaping " to \"

docs reference latest stable version (1.10.0)

Refactor pull request review (#8954)

* refactor submit review

* remove unnecessary code

* remove unused comment

* fix lint

* remove duplicated actions

* remove duplicated actions

* fix typo

* fix comment content

Backport the backport of 1.9.6 (#8970)

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

Fix intermittent GPG Git test failure (#8968)

Add CDNJS to script-src CSP (#8966)

Don't send notification on pending reviews (#8943)

Implement documentation search (#8937)

* Implement documentation search

Signed-off-by: jolheiser <john.olheiser@gmail.com>
Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>

Move index.js to web_src and use webpack to pack them (#8598)

* Move index.js to web_src and use webpack

* Fix initHeatMap

* update eslint and move webpack to devDependencies

* update index.js

* add eslint env node

[skip ci] Updated translations via Crowdin

add /etc/timezone and /etc/localtime volumes (#8911)

Passing these volumes to the container enable gitea to use hosts localtime and timezone.
This is a mandatory change when using the "Setup fail2ban" Guide, but it is not described in the documentation.

[skip ci] Updated translations via Crowdin

Add Close() method to gogitRepository (#8901)

In investigating #7947 it has become clear that the storage component of go-git repositories needs closing.

This PR adds this Close function and adds the Close functions as necessary.

In TransferOwnership the ctx.Repo.GitRepo is closed if it is open to help prevent the risk of multiple open files.

Fixes #7947

Convert EOL to UNIX-style to render MD properly (#8925)

* Convert EOL to UNIX-style to render MD properly

* Update modules/markup/markdown/markdown.go

Co-Authored-By: zeripath <art27@cantab.net>
* Fix lint optimization

* Check for empty content before conversion

* Update modules/util/util.go

Co-Authored-By: zeripath <art27@cantab.net>
* Improved checks and tests

* Add paragraph render test

* Improve speed even more, improve tests

* Small improvement by @gary-kim

* Fix test for DOS

* More improvements

* Restart CI

Show repository size in repo home page and settings (#8940)

* feat: display repository size on repo home and settings

* refactor: move repo size up in setting page

Fix Notify Create Ref Error on tag creation (#8936)

* Fix Notify Create Ref Error on tag creation

* Just use the provided full ref instead of BranchCommit

[skip ci] Updated translations via Crowdin

Update Github Migration Tests (#8938)

Update all Github migration tests to use a new repo created just for
these tests that won't accidentially be modified by regular users
interacting with issues.

Fixes #8895

Fix spelling as suggested by @rmbleeker (#8934)

Move less from public/ to web_src/ so that it will not be packe… (#8908)

[skip ci] Updated translations via Crowdin

Improve notification (#8835)

* Improve notifications

* batch load user

* Update notification only when read

* Fix reorder

* fix lint

* fix test

* fix lint

* make function meaningful

* fix comment

[skip ci] Updated translations via Crowdin

js: Revert change to click behaviour in forceSelection (#8921)

Calling .click() in forceSelection() in blur() causes a focus loop,
this needs a proper fix later but for now just revert the change.

[skip ci] Updated translations via Crowdin

Template Repositories (#8768)

* Start work on templates

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

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Fix IsTemplate vs IsGenerated

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Fix tabs vs spaces

* Tabs vs Spaces

* Add templates to API & start adding tests

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Fix integration tests

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Remove unused User

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Move template tests to existing repos

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Minor re-check updates and cleanup

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

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

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

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

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Test fixes and icon change

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Add new user and repo for tests

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Fix tests (finally)

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Update meta repo with env variables

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Move generation to create page

Combine with repo create template
Modify API search to prioritize owner for repo

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Fix tests and coverage

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Fix swagger and JS lint

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Fix API searching for own private repos

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

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Fix repo search test. User had a private repo that didn't show up

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Another search test fix

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Clarify git content

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>
* Feedback updates

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Add topics WIP

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Finish adding topics

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

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

Migrate temp_repo.go to use git.NewCommand  (#8918)

This PR migrates temp_repo.go to use git.NewCommand instead creating processes by itself - this fixes the problem underlying PR #8905.

There are other places that run git outside of the controlled locale defined in #8548 but temp_repo.go is the only cause of failure of local testing in cases where English is not the default - implying that error messages from those other commands are not interpreted.

Replaces #8905

Only view branch or tag if it match refType requested. (#8899)

* only view branch or tag if it match refName.

* remove pointer in method

Move notifywatchers from models to notification (#8907)

Fix typo in doc (#8914)

Fix typo in doc fail2ban-setup.md

Stop using git count-objects and use raw directory size for repository (#8848)

* Migrate from git count-objects to a raw directory size
* As per @guillep2k ignore unusual files

Rephrase comment about RuntimeDirectory option (#8912)

Auto-subscribe user to repository when they commit/tag to it (#7657)

* Add support for AUTO_WATCH_ON_CHANGES and AUTO_WATCH_ON_CLONE

* Update models/repo_watch.go

Co-Authored-By: Lauris BH <lauris@nix.lv>
* Round up changes suggested by lafriks

* Added changes suggested from automated tests

* Updated deleteUser to take RepoWatchModeDont into account, corrected inverted DefaultWatchOnClone and DefaultWatchOnChanges behaviour, updated and added tests.

* Reinsert import "github.com/Unknwon/com" on http.go

* Add migration for new column `watch`.`mode`

* Remove serv code

* Remove WATCH_ON_CLONE; use hooks, add integrations

* Renamed watch_test.go to repo_watch_test.go

* Correct fmt

* Add missing EOL

* Correct name of test function

* Reword cheat and ini descriptions

* Add update to migration to ensure column value

* Clarify comment

Co-Authored-By: zeripath <art27@cantab.net>
* Simplify if condition

Adjust error reporting from merge failures and use LC_ALL=C for git (#8548)

There are two major components to this PR:

* This PR handles merge and rebase failures from merging a little more nicely with Flash errors rather a 500.
* All git commands are run in the LC_ALL="C" environment to ensure that error messages are in English. This DefaultLocale is defined in a way that if necessary (due to platform weirdness) it can be overridden at build time using LDFLAGS="-X "code.gitea.io/gitea/modules/git.DefaultLocale=C"" with C changed for the locale as necessary.

Fix API Bug (fail on empty assignees) (#8873)

* keep sure if assigneeIDs == nil -> do nothing

* fix #8872

* Revert "keep sure if assigneeIDs == nil -> do nothing"

-> go handle it itself preaty well

This reverts commit e72d94129c4666d5151f6131cb2f8c1de127d9d0.

* clarity comparson

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>
* simplify

* Update models/issue_assignees.go

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>
* Update issue_assignees.go

* simplify more

* add --if oneAssignee != ""-- again

* Update models/issue_assignees.go

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>
* CI.restart()

* Update issue_assignees.go

* add Test for GetUserIDsByNames

* add Test for MakeIDsFromAPIAssigneesToAdd

* fix test

Move code.gitea.io/gitea/routers/api/v1/convert to code.gitea.io/gitea/modules/convert (#8892)

* Move code.gitea.io/gitea/routers/api/v1/convert to code.gitea.io/gitea/modules/convert

* fix fmt

Fix issue with user.fullname (#8902)

Enable punctuations ending mentions (#8889)

* Enable punctuations ending mentions

* Improve tests

Update Github migration test (#8893)

* Update Github migration test

Earlier today #716 was reopened which updated the modification time for
an old milestone (1.6.0) that we use in testing with the assumption that
it is old and won't change. This breaks all builds now, so remove this
test since we have others that test the same code and this milestone
will likely be updated again as that issue changes etc...

* ci

Leave non-dated issues for last (#8871)

[skip ci] Updated translations via Crowdin

Fix require external registration password (#8885)

* Fix require external registration password

* Fix ctx on error condition by @jolheiser

[skip ci] Updated translations via Crowdin

Fix password complexity check on registration (#8887)

* Fix registration password complexity

* Fix integration to use a complex password ;)

Allow to add and remove all repositories to/from team. (#8867)

* Allow to add and remove all repositories to team.

* Change style, buttons on same row.

* Apply suggestions from code review

Grammar

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>
* Move set num repos to lower function.

* Make general language sentences

Create third-party-tools.zh-cn.md (#6303)

* Create third-party-tools.zh-cn.md

* Update third-party-tools.zh-cn.md

* Update third-party-tools.zh-cn.md

Fix bug when migrate from API (#8631)

* fix bug when migrate from API

* fix test

* fix test

* improve

* fix error message

Webhook support custom proxy (#8760)

* Webhook support custom proxy

* Add glob support on webhook proxy host rules

* fix app.ini.sample

* improve code and app.ini.sample

* update cheetsheet about added webhook options

Move some actions to notification/action (#8779)

* Move some actions to notification/action

* Fix test

* fix test

Add password reset to FAQ (#8883)

* Add password reset to FAQ

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Add links to email setup docs

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Add en-us to email setup

Signed-off-by: jolheiser <john.olheiser@gmail.com>
* Update docs/content/doc/help/faq.en-us.md

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>
* Manually merge changes from @guillep2k and add small changes

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

a11y: Improve accessibility of dropdown menus (#8638)

* js: Import Semantic-UI's dropdown.js (version 2.3.1)

* js: Set tabindex=-1 on dropdown items

Setting tabindex=-1 on focusable elements within dropdown menus allows
the user to treat dropdown menus as a single focusable item with its own
internal navigation using arrow keys.

* js: Don't use jQuery to click menu items

Menu items are often <a> elements, which jQuery refuses to trigger click
events on. Instead it just bubbles up to the menu.

Using HTMLElement's click method fixes this and makes menu items
clickable from the keyboard using dropdown menus.

* js: Set correct ARIA 1.1 roles on dropdown menus

Setting role= makes assistive technology aware there is a widget here.
In this case, Orca will now exit browse mode and allow us to capture
keydown events when focused on a dropdown menu. It will also inform the
user that there's a menu focused.

Since dropdowns can be used in multiple elements each with different
ARIA roles, a guessRole method is used to find the correct role.
All roles I consider possible are listed, but only menu is implemented.

* js: Set aria-expanded when dropdown menus show and hide

This is deliberately done before the transition finishes so that screen
readers get immediate feedback.

* js: Set aria-label or aria-labelledby on dropdown menus

This makes dropdown menu buttons screen reader accessible.

aria-labelledby refers to an element using an ID, so the chosen labels
are now assigned a unique ID- This ID is not stable, do not refer to it
with user scripts.

* js: Set aria-activedescendant on dropdown menus

As the menus grab focus and navigate by tracking a 'selected' div class,
assistive technology has no idea that what the current selection is.

Assign IDs to each menu item and set aria-activedescendant to the ID of
the currently selected menu item.

When the menu is unfocused, remove aria-activedescendant- This isn't
neccessary but in my experience it triggers Orca to remind the user of
their current selection when re-focusing the menu.

* Makefile: Make eslint ignore semantic.dropdown.js

This file is taken from Semantic UI which isn't linted upstream.
Ignore it as we won't fix these issues.

* js: Add version note to semantic.dropdown.js

* Add Md5 AppVer to templates/base/footer.tmpl

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>
* Add Md5 AppVer to templates/pwa/serviceworker_js.tmpl

Co-Authored-By: guillep2k <18600385+guillep2k@users.noreply.github.com>
* semantic.dropdown.js -> semantic.dropdown.custom.js

* Use eslintignore

* remove bogus submodule

Fix edit content button on migrated issue content (#8877)

Typo on a closing span tag caused edit button not to work properly on
the original issue content for a migrated issue.

Fixes #8876

Show due date in dashboard issues list (#8860)

* Show due date in dashboard issues list

Include due date when vieiwiing all issues on dashboard (matching what
we show for repo issue lists).

Fixes #8859

* Put in same order as repo issue list

* Add suggested changes and also update repo issue list to match