summaryrefslogtreecommitdiff
path: root/app/services
Commit message (Collapse)AuthorAgeFilesLines
* Refactor web editorrefactor-web-editorDmitriy Zaporozhets2015-06-054-117/+84
| | | | | | | | | | * fix problem with editing non-master branch * before commit make sure branch exists * dont allow user change file in one branch and commit to another existing branch * remove a lot of code duplication * remove outdated statellite errors Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Skip repo removing whem remove user or groupDmitriy Zaporozhets2015-06-033-7/+11
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Merge branch 'repo-remove' into fix-group-removeDmitriy Zaporozhets2015-06-031-13/+52
|\ | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> Conflicts: spec/features/projects_spec.rb
| * Fix tests and increase delay time before remove repositoryDmitriy Zaporozhets2015-06-031-1/+1
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * Move repository when project is removedDmitriy Zaporozhets2015-06-031-13/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ths commit does next: * When we remove project we move repository to path+deleted.git * Then we schedule removal of path+deleted with sidekiq * If repository move failed we abort project removal This should help us with NFS issue when project get removed but repository stayed. The full explanation of problem is below: * rm -rf project.git * rm -rf removes project.git/objects/foo * NFS server renames foo to foo.nfsXXXX because some NFS client (think * Unicorn) still has the file open * rm -rf exits, but project.git/objects/foo.nfsXXX still exists * Unicorn closes the file, the NFS client closes the file (foo), and the * NFS server removes foo.nfsXXX * the directory project.git/objects/ still exists => problem So now we move repository and even if repository removal failed Repository directory is moved so no bugs with project removed but repository directory taken. User still able to create new project with same name. From administrator perspective you can easily find stalled repositories by searching `*+deleted.git` Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Remove projects before group/user. Remove namespace directory asyncfix-group-removeDmitriy Zaporozhets2015-06-032-0/+10
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Wrap group removal into serviceDmitriy Zaporozhets2015-06-031-0/+11
|/ | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Make web editor work correctly after switch from satellitesDmitriy Zaporozhets2015-06-025-5/+7
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Remove files in web editor using ruggedDmitriy Zaporozhets2015-06-021-6/+7
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Create activity event and execute hooks on web editor commitDmitriy Zaporozhets2015-06-023-3/+11
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Use rugged in web editor for base64 encodingDmitriy Zaporozhets2015-06-012-34/+28
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix adding new file to empty repoDmitriy Zaporozhets2015-06-011-1/+1
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Create and edit files in web editor via ruggedDmitriy Zaporozhets2015-06-012-14/+34
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Disable changing of the source branch in merge request update APIStan Hu2015-05-291-1/+2
|
* Merge branch 'support-edit-target-branch-in-mr' into 'master'Dmitriy Zaporozhets2015-05-293-1/+37
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support editing target branch of merge request ### What does this MR do? This MR makes it possible to edit the target branch of a merge request and adds a system note when this happens. ### Why was this MR needed? Because lots of people requested this feature. :) ### Screenshots **Edit MR page** ![image](https://gitlab.com/gitlab-org/gitlab-ce/uploads/9b3d405bf7b5f945e35bae3534c2b67b/image.png) **New MR page** ![image](https://gitlab.com/gitlab-org/gitlab-ce/uploads/3657a2a9efad6d10e8470637d1166bdb/image.png) **System note** ![image](https://gitlab.com/gitlab-org/gitlab-ce/uploads/cc8066f3d3bdf09c0cce27193210567d/image.png) ### What are the relevant issue numbers? * Closes https://github.com/gitlabhq/gitlabhq/issues/7105 * See: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/130! See merge request !738
| * Support editing target branch of merge requestStan Hu2015-05-293-1/+37
| | | | | | | | | | Closes https://github.com/gitlabhq/gitlabhq/issues/7105 See: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/130
* | You can not remove user if he/she is an only owner of groupDmitriy Zaporozhets2015-05-281-0/+10
|/ | | | | | | To prevent loose of group data you need to transfer or remove group first before you can remove user Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Merge branch 'rs-to_reference' into 'master'Dmitriy Zaporozhets2015-05-271-42/+18
|\ | | | | | | | | | | | | | | Add to_reference method to referable models Now there is a single source of information for which attribute a model uses to be referenced, and its special character. See merge request !641
| * More SystemNoteService cleanuprs-to_referenceRobert Speicher2015-05-261-42/+18
| |
* | Add a note when an Issue or Merge Request's title changesRobert Speicher2015-05-264-0/+32
|/
* Add support for Webhook note eventsStan Hu2015-05-211-1/+1
| | | | Closes https://github.com/gitlabhq/gitlabhq/issues/6745
* Fix.Douwe Maan2015-05-151-1/+1
|
* Move stuff around a bit in NotifictionService.Douwe Maan2015-05-151-8/+15
|
* Call merge request web hook for each commitsArthur Gautier2015-05-131-0/+15
| | | | | | | Call merge request web hook every times a new commit hits the source branch Signed-off-by: Arthur Gautier <baloo@gandi.net>
* Refactor SystemNoteService.cross_reference_disallowed?Robert Speicher2015-05-111-6/+13
|
* Update SystemNoteService method naming conventionsRobert Speicher2015-05-116-77/+73
| | | | | Now the verb comes first, and there is no restriction on singular/plural.
* Remove legacy special case for emphasized reference notesRobert Speicher2015-05-111-13/+9
|
* Add a SystemNoteService classRobert Speicher2015-05-116-8/+313
| | | | | There's a lot of code in the Note model that only deals with creating system notes, so we're going to split that into its own class.
* Added X-GitLab-Event header for web hooksbugagazavr2015-05-082-4/+4
|
* Make the first branch pushed to an empty repository the default HEAD.Stan Hu2015-05-051-0/+4
| | | | | | | | | | | In an empty repository, pushing a new branch not called "master" would leave HEAD in an unknown state, causing ambiguity if another branch were pushed. This could in turn cause a new protected branch to be created and cause the default branch to change. * Closes #1561 * Closes #1576 * Closes https://github.com/gitlabhq/gitlabhq/issues/8883
* Revert "Added X-GitLab-Event header for web hooks"Valery Sizov2015-04-272-4/+4
| | | | This reverts commit 548f182814acd0f7a110e6c165c186e345901b00.
* Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ceDmitriy Zaporozhets2015-04-272-53/+21
|\
| * Use Projects::CreateService to fork projects so that after-create hooks are run.forked-eventDouwe Maan2015-04-242-53/+21
| |
* | Added X-GitLab-Event header for web hooksbugagazavr2015-04-252-4/+4
|/
* Link cross-project cross-reference notes to correct project.Douwe Maan2015-04-243-4/+3
|
* No longer needed to pass project argument to commit methods.Douwe Maan2015-04-243-13/+9
|
* Use project.commit convenience method.Douwe Maan2015-04-243-3/+3
|
* Let commit model know about its project.Douwe Maan2015-04-241-1/+1
|
* Clean up code somewhat.better-commit-mentionsDouwe Maan2015-04-171-13/+15
|
* Only send note notifications to people accessible by the note author.Douwe Maan2015-04-151-2/+2
| | | | Should not make a difference most of the time, since if they're participating in the thread, they have access to the project.
* Clean up code around commit mentions.Douwe Maan2015-04-152-10/+5
|
* Fix for snippets.fix-mention-notificationDouwe Maan2015-04-151-2/+4
|
* Fix Mention notification level.Douwe Maan2015-04-151-24/+13
|
* Let invites be declined.Douwe Maan2015-04-141-0/+8
|
* Add emails around invitation.Douwe Maan2015-04-141-0/+16
|
* Track who created a group or project member.Douwe Maan2015-04-142-2/+2
|
* Merge branch 'reference-access-control' into 'master'Dmitriy Zaporozhets2015-04-133-29/+33
|\ | | | | | | | | | | | | | | Only allow users to reference groups, projects, issues, MRs, commits they have access to. Addresses https://dev.gitlab.org/gitlab/gitlabhq/issues/2183. See merge request !1742
| * Fix errors.Douwe Maan2015-04-021-12/+7
| |
| * Only allow users to cross-reference and close issues they have access to.Douwe Maan2015-04-021-2/+2
| |
| * Only allow user to see participants from groups they have access to.Douwe Maan2015-04-021-2/+2
| |