summaryrefslogtreecommitdiff
path: root/app/services/groups
Commit message (Collapse)AuthorAgeFilesLines
* Fix snippets API not working with visibility levelsh-fix-snippet-visibility-apiStan Hu2019-08-281-4/+0
| | | | | | | | | | | | | | | | | | | | When a restricted visibility level of `private` is set in the instance, creating a snippet with the `visibility` level would always fail. This happened because: 1. `params[:visibility]` was a string (e.g. "public") 2. `CreateSnippetService` and `UpdateSnippetService` only looked at `params[:visibility_level]`, which was `nil`. To fix this, we: 1. Make `CreateSnippetService` look at the newly-built `snippet.visibility_level`, since the right value is assigned by the `VisibilityLevel#visibility=` method. 2. Modify `UpdateSnippetService` to handle both `visibility_level` and `visibility` parameters. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/66050
* Allow disabling group/project email notificationsBrett Walker2019-08-151-0/+5
| | | | | | - Adds UI to configure in group and project settings - Removes notification configuration for users when disabled at group or project level
* Remove code related to object hierarchy in MySQLremove-nested-groups-checksHeinrich Lee Yu2019-07-252-5/+0
| | | | | These are not required because MySQL is not supported anymore
* Upgrade to Rails 5.2upgrade-rails-5-2-ceHeinrich Lee Yu2019-07-121-2/+1
| | | | Updates changed method names and fixes spec failures
* Add a rubocop for Rails.loggerMayra Cabrera2019-07-101-1/+1
| | | | | | Suggests to use a JSON structured log instead Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/54102
* Merge branch 'forbid-the-usage-of-reload' into 'master'Lin Jen-Shin2019-04-151-1/+1
|\ | | | | | | | | | | | | Forbid the usage of reload Closes #60218 See merge request gitlab-org/gitlab-ce!27125
| * Forbid the use of `#reload` and prefer `#reset`forbid-the-usage-of-reloadKamil Trzciński2019-04-151-1/+1
| | | | | | | | | | | | | | The `#reload` makes to load all objects into memory, and the main purpose of `#reload` is to drop the association cache. The `#reset` seems to solve exactly that case.
* | Externalize strings detected by rubocop-i18nMartin Wortschack2019-04-152-8/+8
|/ | | | | | - Externalize strings in milestones_helper - Externalize strings in app/services - Update PO file
* Squashed commits and modified visibility level outputSara Ahbabou2019-04-121-1/+5
| | | | Added changelog and rebased
* Backport some changes from EE3314-add-on-runner-minutes-for-gitlab-com-ceRubén Dávila2019-04-023-0/+9
| | | | MR: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/9815
* Enable/disable Auto DevOps at Group levelMayra Cabrera2019-03-121-0/+17
| | | | | | | | | | | | - Includes instance methods on Group model to detect when a group has AutoDevOps explicitly/implicitly enabled/disabled. - Includes migration to add a new column to namespaces table - Add UI necessary modifications - Add service and controller to update auto devops related instances - Updates project and groups auto devops badges Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/52447
* Merge branch 'fix-group-without-owner' into 'master'Sean McGivern2019-03-071-0/+10
|\ | | | | | | | | | | | | fix group without owner after transfer Closes #58086 See merge request gitlab-org/gitlab-ce!25573
| * fix group without owner after transferPeter Marko2019-03-011-0/+10
| |
* | Adds the Rubocop ReturnNil copAndrew Newdigate2019-03-061-1/+1
|/ | | | | This style change enforces `return if ...` instead of `return nil if ...` to save maintainers a few minor review points
* Merge branch 'fix-misspellings-app-comments' into 'master'Rémy Coutable2019-02-262-2/+2
| | | | | Fix misspellings in app/spec comments See merge request gitlab-org/gitlab-ce!25517
* Revert "Merge branch 'fix-misspellings-app-comments' into 'master'"Michael Kozono2019-02-252-2/+2
| | | | | This reverts commit 9202bbd129537a698b986e6295d0c783b5a84815, reversing changes made to 4b282e9ce1ae246c4538b3ede18d1380ea778029.
* Fix misspellings in app/spec commentsTakuya Noguchi2019-02-252-2/+2
| | | | Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
* Reduce diff with EE in Groups::UpdateServiceRémy Coutable2019-02-041-0/+6
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Reduce diff with EE in Groups::CreateServiceRémy Coutable2019-02-011-0/+6
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Merge branch 'security-todos_not_redacted_for_guests' into 'master'John Jarvis2019-01-021-1/+1
|\ | | | | | | | | [master] Security todos not redacted for guests See merge request gitlab/gitlabhq!2697
| * Delete confidential issue todos for guestsFelipe Artur2018-12-141-1/+1
| | | | | | | | | | Fix leaking information of confidential issues on TODOs when user is downgraded to guest access.
* | Rename GroupHierarchy into ObjectHierarchyJarka Košanová2018-12-192-2/+2
| | | | | | | | | | | | - we now use the hierarchy class also for epics - also rename supports_nested_groups? into supports_nested_objects? - move it to a concern
* | Change deprecated `except!` usagesRobert Speicher2018-12-161-1/+1
|/ | | | | These can just be `delete` calls, with the caveat that it only takes one argument.
* Fix Groups::UpdateService#execute not returning correct error codeStan Hu2018-10-181-2/+4
| | | | | This was causing problems in EE, where audit events were being generated even if the project failed to save.
* Disable existing offenses for the CodeReuse copsYorick Peterse2018-09-112-0/+6
| | | | | This whitelists all existing offenses for the various CodeReuse cops, of which most are triggered by the CodeReuse/ActiveRecord cop.
* don't trigger project deletion hooks twice when removing a groupGabriel Mazetto2018-08-261-1/+4
|
* Fix: Project deletion may not log audit events during group deletion49796-project-deletion-may-not-log-audit-events-during-group-deletionValery Sizov2018-08-161-3/+4
|
* Remove group todos when a users looses accessJarka Kadlecová2018-08-021-1/+10
|
* Enable frozen string in apps/services/**/*.rbrepo-forks/gitlab-ce-frozen-string-enable-apps-services-innergfyoung2018-07-166-0/+12
| | | | | | For directories application_settings --> labels. Partially addresses #47424.
* Add manifest import featureDmitriy Zaporozhets2018-07-111-5/+7
| | | | | | | | | It allows user to automatically import multiple repositories with nested structure by uploading a manifest xml file. AOSP project was used as an example during development of this feature. Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Merge branch 'fix/gh-namespace-issue' into 'security-10-4'James Lopez2018-02-091-3/+7
| | | | [10.4] Fix GH namespace security issue
* 31885 - Ability to transfer a single group to another groupMayra Cabrera2018-02-061-0/+96
|
* Remove soft removals related codeYorick Peterse2018-01-081-2/+1
| | | | | | | | | | | | | | This removes all usage of soft removals except for the "pending delete" system implemented for projects. This in turn simplifies all the query plans of the models that used soft removals. Since we don't really use soft removals for anything useful there's no point in keeping it around. This _does_ mean that hard removals of issues (which only admins can do if I'm not mistaken) can influence the "iid" values, but that code is broken to begin with. More on this (and how to fix it) can be found in https://gitlab.com/gitlab-org/gitlab-ce/issues/31114. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/37447
* Make Members with Owner and Master roles always able to create subgroups30473-allow-creation-of-subgroups-with-gitlab_default_can_create_group-set-to-falseRuben Davila2017-09-071-9/+29
|
* Refer to “Share with group lock” consistentlyimprove-share-locking-feature-for-subgroupsMichael Kozono2017-09-061-1/+1
|
* Make UpdateService error message translatableWinnie Hellmann2017-09-061-1/+1
|
* Refactor based on code reviewMichael Kozono2017-09-061-1/+1
|
* Dry up method for code climateMichael Kozono2017-09-061-10/+3
|
* Enforce share_with_group_lock rulesMichael Kozono2017-09-061-1/+17
| | | | …in Groups::UpdateService instead of only in the browser.
* Handle creating a nested group on MySQL correctlybvl-fix-mysql-bare-repository-importerBob Van Landuyt2017-08-251-0/+4
| | | | | Since we don't support nested groups on MySQL, raise an error explaining that on import instead of trying anyway.
* Migrate creation of nested groups into a serviceBob Van Landuyt2017-08-231-0/+45
|
* Merge branch '35845-improve-subgroup-creation-permissions' into 'master'Douwe Maan2017-08-221-2/+2
|\ | | | | | | | | | | | | Improves subgroup creation permissions Closes #35845 See merge request !13418
| * Improves subgroup creation permissions35845-improve-subgroup-creation-permissionsTiago Botelho2017-08-211-2/+2
| |
* | Make sure repository's removal work for legacy and hashed storages28283-uuid-storageGabriel Mazetto2017-08-221-1/+1
|/
* Remove Mattermost team with GitLab groupZ.J. van de Weg2017-07-281-0/+2
| | | | | | | When destroying a group, now an API call is made to the Mattermost server to request the deletion of the project. Actual team deletion on the Mattermost side happens async, so the runtime shouldn't increase by more than a second.
* Defer project destroys within a namespace in ↵Stan Hu2017-06-291-2/+1
| | | | | | | | | | | Groups::DestroyService#async_execute Group#destroy would actually hard-delete all associated projects even though the acts_as_paranoia gem is used, preventing Projects::DestroyService from doing any work. We first noticed this while trying to log all projects deletion to the Geo log.
* Fix current feature related specsTiago Botelho2017-06-281-1/+1
|
* Merge branch 'jej-group-name-disclosure' into 'security'Sean McGivern2017-03-291-0/+8
| | | | | | Prevent private group disclosure via parent_id See merge request !2077
* Fix eslintZ.J. van de Weg2017-03-061-6/+3
|
* Merge branch 'master' into zj-create-mattermost-teamZ.J. van de Weg2017-03-021-1/+2
|\