]> git.openstreetmap.org Git - rails.git/commitdiff
Merge pull request #2427 from mmd-osm/patch/remove_expand_bbox
authorAndy Allan <git@gravitystorm.co.uk>
Wed, 4 Dec 2019 11:48:34 +0000 (12:48 +0100)
committerGitHub <noreply@github.com>
Wed, 4 Dec 2019 11:48:34 +0000 (12:48 +0100)
Remove expand_bbox endpoint

1  2 
app/abilities/api_ability.rb
app/abilities/api_capability.rb
config/routes.rb
test/controllers/api/changesets_controller_test.rb

index 217fe9713c8745f0c666e261389175e883425834,7aef15f114e017b7748cd2fa0185f918c209f4c4..62cd2b17ebbd41ad9408ebcfc5730134991b3506
@@@ -34,10 -34,10 +34,10 @@@ class ApiAbilit
          can [:new, :create], Report
          can [:create, :show, :update, :destroy, :data], Trace
          can [:details, :gpx_files], User
 -        can [:read, :read_one, :update, :update_one, :delete_one], UserPreference
 +        can [:index, :show, :update, :update_all, :destroy], UserPreference
  
          if user.terms_agreed?
-           can [:create, :update, :upload, :close, :subscribe, :unsubscribe, :expand_bbox], Changeset
+           can [:create, :update, :upload, :close, :subscribe, :unsubscribe], Changeset
            can :create, ChangesetComment
            can [:create, :update, :delete], Node
            can [:create, :update, :delete], Way
index 64861f1d679cad8b962618909e86c31cfc9d2742,a4ca252045aebae3bba2e559e71e4a2bbe3963d9..beb4d39bf3fa62ba20641899a89220cb91594b71
@@@ -10,11 -10,11 +10,11 @@@ class ApiCapabilit
        can [:create, :update, :destroy], Trace if capability?(token, :allow_write_gpx)
        can [:details], User if capability?(token, :allow_read_prefs)
        can [:gpx_files], User if capability?(token, :allow_read_gpx)
 -      can [:read, :read_one], UserPreference if capability?(token, :allow_read_prefs)
 -      can [:update, :update_one, :delete_one], UserPreference if capability?(token, :allow_write_prefs)
 +      can [:index, :show], UserPreference if capability?(token, :allow_read_prefs)
 +      can [:update, :update_all, :destroy], UserPreference if capability?(token, :allow_write_prefs)
  
        if token&.user&.terms_agreed?
-         can [:create, :update, :upload, :close, :subscribe, :unsubscribe, :expand_bbox], Changeset if capability?(token, :allow_write_api)
+         can [:create, :update, :upload, :close, :subscribe, :unsubscribe], Changeset if capability?(token, :allow_write_api)
          can :create, ChangesetComment if capability?(token, :allow_write_api)
          can [:create, :update, :delete], Node if capability?(token, :allow_write_api)
          can [:create, :update, :delete], Way if capability?(token, :allow_write_api)
Simple merge