Bug 771411 - Refactor distributions route
Summary: Refactor distributions route
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: WebUI
Version: 6.0.1
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: Unspecified
Assignee: Brad Buckingham
QA Contact: Garik Khachikyan
URL:
Whiteboard:
Depends On:
Blocks: katello-blockers
TreeView+ depends on / blocked
 
Reported: 2012-01-03 16:37 UTC by Lukas Zapletal
Modified: 2019-09-25 21:08 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-22 18:17:01 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 782022 0 unspecified CLOSED UI permissions review 2021-02-22 00:41:40 UTC

Internal Links: 782022

Description Lukas Zapletal 2012-01-03 16:37:08 UTC
Hello,

please change the route

/distributions/:id

to include repository id

/repositories/:repository_id/distributions/:id

Permission check has been prepared and after the change is commited it can be enabled:

ebc2e52a3a5da76d159fd9e41e8d7cfbfbbb62c2

Also unit test is ready - just to be uncommented.

In short:

1) Rename the route
2) Change it in the UI/JavaScript code
3) Add new before filter find_product that will fetch @product for given repo id
4) Uncomment the following line:

  #readable = lambda{ @product.readable? }
  readable = lambda{ true } # delete this one

5) Uncomment unit test

Permissions are implemented now.

Comment 1 Brad Buckingham 2012-01-18 21:38:04 UTC
commit - f4358724cf4400f9812396f23255076b97792fb3

Comment 2 Mike McCune 2012-01-26 19:08:08 UTC
mass ON_QA move

Comment 7 Mike McCune 2013-08-16 18:03:28 UTC
getting rid of 6.0.0 version since that doesn't exist


Note You need to log in before you can comment on or make changes to this bug.