Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Find by fqname of automate class and namespace doesn't work with leading slash #236

Closed
mkanoor opened this issue Jul 21, 2014 · 0 comments · Fixed by #237
Closed

Find by fqname of automate class and namespace doesn't work with leading slash #236

mkanoor opened this issue Jul 21, 2014 · 0 comments · Fixed by #237
Labels
bug

Comments

@mkanoor
Copy link
Contributor

mkanoor commented Jul 21, 2014

The fqname method gives back a fully qualified name of a class or an instance, which includes the domain, namespace, class.
The fqname doesn't contain the leading /

When a user calls MiqAeNamespace.find_by_fqname or MiqAeClass.find_by_fqname if there is a leading slash this call fails. Both of these calls only work if the leading slash has been removed.

This issue was opened to fix the MiqAeNamespace.find_by_fqname and MiqAeClass.find_by_fqname so that it can work with or without the leading slash.
@Fryguy Fryguy added the bug label Jul 21, 2014
@mkanoor mkanoor mentioned this issue Jul 21, 2014
Merged
mkanoor added a commit to mkanoor/manageiq that referenced this issue Jul 29, 2014
@mkanoor
Allow for search with leading / for namespace and classes
abe0c77 
ManageIQ#236
Fix ManageIQ#236
MiqAeNamespace.find_by_fqname works with or without leading slash.
Without leading slash is the recommended way.
@Fryguy Fryguy added this to the Sprint 11 - Aug 19, 2014 milestone Aug 29, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow for search with leading / for namespace and classes mkanoor/manageiq
2 participants
@Fryguy @mkanoor