[obs-commits] [openSUSE/open-build-service] decc89: [api] fix crash on wrong command
Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: decc8982e4fd9ce0d56067995da9236d9ec6dfbd https://github.com/openSUSE/open-build-service/commit/decc8982e4fd9ce0d56067... Author: Adrian Schröter <adrian@suse.de> Date: 2016-12-14 (Wed, 14 Dec 2016) Changed paths: M src/api/app/controllers/request_controller.rb M src/api/test/functional/request_controller_test.rb Log Message: ----------- [api] fix crash on wrong command Commit: 3486a99fbfbda7df31bfb06eee967a7d5b4a0466 https://github.com/openSUSE/open-build-service/commit/3486a99fbfbda7df31bfb0... Author: Adrian Schröter <adrian@suse.de> Date: 2016-12-14 (Wed, 14 Dec 2016) Changed paths: M src/api/app/models/bs_request_action_maintenance_incident.rb M src/api/test/functional/maintenance_test.rb M src/api/test/unit/code_quality_test.rb Log Message: ----------- [api] fix maintenance incident merge code ignored the specified revision of the request on accept. Seems to be the case since quite some time (when code got moved to model) Quite some cleanup since that also leaded to unnecessary usage of method variables also fixed some request number handling (The code became simpler now, but I had to increase the "code quality" number :/) Commit: 0017baf8cdf1bdd3ff0a5518759802cad6dee0cc https://github.com/openSUSE/open-build-service/commit/0017baf8cdf1bdd3ff0a55... Author: Björn Geuken <bgeuken@suse.de> Date: 2016-12-15 (Thu, 15 Dec 2016) Changed paths: M src/api/app/controllers/request_controller.rb M src/api/app/models/bs_request_action_maintenance_incident.rb M src/api/test/functional/maintenance_test.rb M src/api/test/functional/request_controller_test.rb M src/api/test/unit/code_quality_test.rb Log Message: ----------- Merge pull request #2462 from adrianschroeter/maint_inc_revision_fix Maintenance incident revision fix Compare: https://github.com/openSUSE/open-build-service/compare/723594252ef9...0017ba...
participants (1)
-
Björn Geuken