Project

General

Profile

« Previous | Next » 

Revision 107ad431

Added by Brad Buckingham about 10 years ago

Content Views Rework: prepend content_view on filter models, update routes

This commit addresses the following:

1. Rename the filter models/schema, prepending 'content_view_' to each.
This change is to be consistent with other models
(e.g. content_view_environments) as well as to ensure that there is
never a name clash with the core or future models added to katello

2. Update routes to also account for the above change. For example,
rather than define:
GET /filters/:id
defint:
GET /content_view_filters/:id

View differences:

app/controllers/katello/api/v2/package_groups_controller.rb
api :GET, "/package_groups", "List package groups"
api :GET, "/content_views/:content_view_id/filters/:filter_id/package_groups", "List package groups"
api :GET, "/filters/:filter_id/package_groups", "List package groups"
api :GET, "/content_view_filters/:content_view_filter_id/package_groups", "List package groups"
api :GET, "/repositories/:repository_id/package_groups", "List package groups"
param :content_view_id, :identifier, :desc => "content view identifier"
param :filter_id, :identifier, :desc => "content view filter identifier"
param :content_view_filter_id, :identifier, :desc => "content view filter identifier"
param :repository_id, :identifier, :desc => "repository identifier", :required => true
def index
collection = if @repo && !@repo.puppet?
......
@filter = @view.filters.find_by_id(params[:filter_id])
fail HttpErrors::NotFound, _("Couldn't find Filter with id=%s") % params[:filter_id] unless @filter
else
@filter = Filter.find(params[:filter_id]) if params[:filter_id]
@filter = ContentViewFilter.find(params[:content_view_filter_id]) if params[:content_view_filter_id]
end
end

Also available in: Unified diff