Raise NotImplementedError instead of NotImplemented
Bug #1339855 reported by
Christian Berendt
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
CloudCafe |
Fix Committed
|
Undecided
|
Unassigned | ||
Fuel for OpenStack |
Fix Committed
|
Medium
|
Christian Berendt | ||
OpenStack Heat |
Fix Released
|
Undecided
|
Ji.Wei | ||
OpenStack Identity (keystone) |
Fix Released
|
Medium
|
Ji.Wei | ||
Solar |
In Progress
|
Undecided
|
Unassigned | ||
Warm |
Invalid
|
Undecided
|
Unassigned | ||
keystonemiddleware |
Fix Released
|
Undecided
|
Jamie Lennox | ||
neutron |
Fix Released
|
Undecided
|
Christian Berendt |
Bug Description
NotImplementedError is the name of the exception (https:/
NotImplemented is the name of a constant (https:/
It makes no sense to raise a constant. The exception should be raised instead.
Changed in neutron: | |
assignee: | nobody → Christian Berendt (berendt) |
status: | New → In Progress |
Changed in fuel: | |
importance: | Undecided → Medium |
milestone: | none → 5.1 |
tags: | added: icehouse-backport-potential |
Changed in neutron: | |
milestone: | none → juno-3 |
status: | Fix Committed → Fix Released |
Changed in neutron: | |
milestone: | juno-3 → 2014.2 |
Changed in keystone: | |
importance: | Undecided → Medium |
assignee: | nobody → Ji.Wei (jiwei) |
Changed in warm: | |
assignee: | nobody → Ji.Wei (jiwei) |
Changed in solar: | |
assignee: | nobody → Ji.Wei (jiwei) |
Changed in keystonemiddleware: | |
assignee: | nobody → Ji.Wei (jiwei) |
status: | New → In Progress |
Changed in solar: | |
status: | New → In Progress |
Changed in keystone: | |
milestone: | none → newton-rc1 |
Changed in heat: | |
assignee: | nobody → Ji.Wei (jiwei) |
Changed in heat: | |
status: | New → In Progress |
Changed in keystonemiddleware: | |
assignee: | Ji.Wei (jiwei) → Jamie Lennox (jamielennox) |
Changed in warm: | |
assignee: | Ji.Wei (jiwei) → nobody |
Changed in solar: | |
assignee: | Ji.Wei (jiwei) → nobody |
To post a comment you must log in.
Fix proposed to branch: master /review. openstack. org/105858
Review: https:/