- 09 Dec, 2016 1 commit
-
-
Douwe Maan authored
Replace MR access checks with use of MergeRequestsFinder Split from !2024 to partially solve https://gitlab.com/gitlab-org/gitlab-ce/issues/23867
- Potentially untested - No test coverage - Test coverage of some sort exists (a test failed when error raised) - Test coverage of return value (a test failed when nil used) - Permissions check tested - [x] app/finders/notes_finder.rb:17 - [x] app/views/layouts/nav/_project.html.haml:80 [`.count`] - [x] app/controllers/concerns/creates_commit.rb:84 - [x] app/controllers/projects/commits_controller.rb:24 - [x] app/controllers/projects/compare_controller.rb:56 - [x] app/controllers/projects/discussions_controller.rb:29 - [x] app/controllers/projects/todos_controller.rb:27 - [x] app/models/commit.rb:268 - [x] lib/gitlab/search_results.rb:71 - [x] https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2024/diffs#d1c10892daedb4d4dd3d4b12b6d071091eea83df_267_266 Memoize ` merged_merge_request(current_user)` - [x] https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2024/diffs#d1c10892daedb4d4dd3d4b12b6d071091eea83df_248_247 Expected side effect for `merged_merge_request!`, consider `skip_authorization: true`. - [x] https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2024/diffs#d1c10892daedb4d4dd3d4b12b6d071091eea83df_269_269 Scary use of unchecked `merged_merge_request?` See merge request !2033
-
- 24 Oct, 2016 1 commit
-
-
Lin Jen-Shin authored
Closes #23615
-
- 20 Oct, 2016 1 commit
-
-
Valery Sizov authored
-
- 29 Apr, 2016 1 commit
-
-
Paco Guzman authored
- Parameter validation as ISO8601 format
-
- 10 Feb, 2016 1 commit
-
-
Douglas Barbosa Alexandre authored
-
- 07 Jan, 2016 2 commits
-
-
Dmitriy Zaporozhets authored
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
Dmitriy Zaporozhets authored
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
- 28 Dec, 2015 1 commit
-
-
Robert Speicher authored
-
- 22 Oct, 2015 1 commit
-
-
Valery Sizov authored
-
- 20 Apr, 2015 1 commit
-
-
Jeroen van Baarsen authored
Signed-off-by: Jeroen van Baarsen <jeroenvanbaarsen@gmail.com>
-
- 20 Feb, 2015 1 commit
-
-
Douwe Maan authored
-
- 06 Feb, 2015 1 commit
-
-
Dmitriy Zaporozhets authored
-
- 26 Jan, 2015 1 commit
-
-
Dmitriy Zaporozhets authored
-
- 05 Nov, 2014 1 commit
-
-
Valery Sizov authored
-
- 13 Oct, 2014 1 commit
-
-
Ciro Santilli authored
with existing method_missing. Pattern already used extensively, so let's be consistent and use it everywhere.
-
- 07 Oct, 2014 1 commit
-
-
Ciro Santilli authored
-
- 10 Jun, 2014 1 commit
-
-
skv authored
-
- 28 Nov, 2013 1 commit
-
-
Dmitriy Zaporozhets authored
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
- 23 Jun, 2013 1 commit
-
-
Dmitriy Zaporozhets authored
-
- 31 Mar, 2013 2 commits
-
-
Dmitriy Zaporozhets authored
-
Dmitriy Zaporozhets authored
-
- 01 Mar, 2013 1 commit
-
-
Dmitriy Zaporozhets authored
-
- 03 Jan, 2013 1 commit
-
-
Dmitriy Zaporozhets authored
-
- 27 Sep, 2012 1 commit
-
-
Cyril authored
-
- 26 Sep, 2012 6 commits
-
-
Robert Speicher authored
-
Robert Speicher authored
Takes tree-ish + path as ID
-
Robert Speicher authored
-
Robert Speicher authored
-
Robert Speicher authored
Commits#show (plural) is going to be for showing commit history on a specific path.
-
Cyril authored
-
- 21 Sep, 2012 1 commit
-
-
Dmitriy Zaporozhets authored
-
- 14 Sep, 2012 1 commit
-
-
Peter Gonda authored
-
- 06 Sep, 2012 1 commit
-
-
Nihad Abbasov authored
-
- 05 Sep, 2012 1 commit
-
-
Robert Speicher authored
That was the only place it was used.
-
- 11 Aug, 2012 1 commit
-
-
randx authored
-
- 10 Aug, 2012 1 commit
-
-
Robert Speicher authored
Didn't bother with files in db/, config/, or features/
-
- 25 Jul, 2012 1 commit
-
-
randx authored
-
- 22 Jul, 2012 1 commit
-
-
Riyad Preukschas authored
-
- 20 Jul, 2012 1 commit
-
-
Dmitriy Zaporozhets authored
Controllers refactoring with contexts Move commit compare logic to model
-
- 28 Jun, 2012 1 commit
-
-
randx authored
-