Audit & action plans api controllers - unused parameter in get_all() and detail()

Bug #1534615 reported by Vincent Françoise on 2016-01-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
watcher
Medium
Gábor Antal

Bug Description

In the Audit controller (watcher/api/controllers/v1/audit.py), we have an unused "audit_uuid" parameter in both get_all() and detail()

Similarly, the Action Plan controller (watcher/api/controllers/v1/action_plan.py) has an unused "action_plan_uuid" parameter in both get_all() and detail().

We need to remove them as they are useless

Tags: api Edit Tag help
Changed in watcher:
importance: Undecided → Medium
Changed in watcher:
status: New → Confirmed
Changed in watcher:
assignee: nobody → Gábor Antal (gabor.antal)
Changed in watcher:
milestone: none → mitaka-2

Fix proposed to branch: master
Review: https://review.openstack.org/269158

Changed in watcher:
status: Confirmed → In Progress

Reviewed: https://review.openstack.org/269158
Committed: https://git.openstack.org/cgit/openstack/watcher/commit/?id=58fb86a3dae82efd0e6f1bdad7a8c1a2b787f1f3
Submitter: Jenkins
Branch: master

commit 58fb86a3dae82efd0e6f1bdad7a8c1a2b787f1f3
Author: Gábor Antal <email address hidden>
Date: Mon Jan 18 16:55:18 2016 +0100

    Removed unused parameters from api controllers

    In the Audit controller (watcher/api/controllers/v1/audit.py),
    we have an unused "audit_uuid" parameter in both get_all()
    and detail()

    Similarly, the Action Plan controller
    (watcher/api/controllers/v1/action_plan.py) has an unused
    "action_plan_uuid" parameter in both get_all() and detail().

    I also removed them from the @wsme_pecan.wsexpose decorator

    Change-Id: I7f1edfd44dd95c9768249650e19b16fcbe916b89
    Closes-Bug: #1534615

Changed in watcher:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers