Standardise on avoiding except lists for check_api_readable

Although this is technically duplicative, it's much easier to read
and therefore to maintain, particularly if you put the _readable one
first.

app/controllers/api/changeset_comments_controller.rb

@@ -1,7 +1,7 @@
 module Api
   class ChangesetCommentsController < ApiController
+    before_action :check_api_readable
     before_action :check_api_writable
-    before_action :check_api_readable, :except => [:create]
     before_action :authorize
 
     authorize_resource

app/controllers/api/changesets_controller.rb

@@ -2,8 +2,8 @@
 
 module Api
   class ChangesetsController < ApiController
+    before_action :check_api_readable
     before_action :check_api_writable, :only => [:create, :update, :upload, :subscribe, :unsubscribe]
-    before_action :check_api_readable, :except => [:index, :create, :update, :upload, :download, :subscribe, :unsubscribe]
     before_action :setup_user_auth, :only => [:show]
     before_action :authorize, :only => [:create, :update, :upload, :close, :subscribe, :unsubscribe]
 

app/controllers/api/nodes_controller.rb

@@ -2,8 +2,8 @@
 
 module Api
   class NodesController < ApiController
+    before_action :check_api_readable
     before_action :check_api_writable, :only => [:create, :update, :delete]
-    before_action :check_api_readable, :except => [:create, :update, :delete]
     before_action :authorize, :only => [:create, :update, :delete]
 
     authorize_resource

app/controllers/api/relations_controller.rb

@@ -1,7 +1,7 @@
 module Api
   class RelationsController < ApiController
+    before_action :check_api_readable
     before_action :check_api_writable, :only => [:create, :update, :delete]
-    before_action :check_api_readable, :except => [:create, :update, :delete]
     before_action :authorize, :only => [:create, :update, :delete]
 
     authorize_resource

app/controllers/api/traces_controller.rb

@@ -1,6 +1,6 @@
 module Api
   class TracesController < ApiController
-    before_action :check_api_readable, :only => [:show, :data]
+    before_action :check_api_readable
     before_action :check_api_writable, :only => [:create, :update, :destroy]
     before_action :set_locale
     before_action :authorize

app/controllers/api/ways_controller.rb

@@ -1,7 +1,7 @@
 module Api
   class WaysController < ApiController
+    before_action :check_api_readable
     before_action :check_api_writable, :only => [:create, :update, :delete]
-    before_action :check_api_readable, :except => [:create, :update, :delete]
     before_action :authorize, :only => [:create, :update, :delete]
 
     authorize_resource