Summary

Add a class for handling the registration and lookup of API resources.

Review Request #7477 — Created July 2, 2015 and submitted July 7, 2015, 4:39 p.m.

Information

Owner

chipx86

Repository

Djblets

Branch

release-0.9.x

Bugs

Depends On

Commit

a3d18a8...

Reviewers

Groups

djblets

People

Description

In Review Board, we had a nice little class that would handle the
loading of resources on-demand, and would allow for resources to
reference each other sanely without worrying about import loops.

This change brings the base of that class into Djblets, so that any
consumer of the API can use it. It's now easy to get this same
functionality in all applications interfacing with djblets.webapi.

Testing Done

Made use of this in Review Board, and all unit tests passed.

Issues

Description	From	Last Updated
undefined name 'logging'	reviewbot	July 6, 2015, 3:55 p.m.
I think we should start converting to using logging.exception here. Once that's changed, this should also pass in name as …	david	July 6, 2015, 3:55 p.m.

Tool: PEP8 Style Checker
Processed Files:
    djblets/webapi/resources/registry.py



Tool: Pyflakes
Processed Files:
    djblets/webapi/resources/registry.py

djblets/webapi/resources/registry.py (Diff revision 1)
The issue has been resolved. Show all issues
```
 undefined name 'logging'
```

djblets/webapi/resources/registry.py (Diff revision 1)

The issue has been resolved. Show all issues

I think we should start converting to using logging.exception here. Once that's changed, this should also pass in name as an argument rather than formatting it in.

Change Summary:


Added a missing import.
Switched to using logging.exception, which also improves this by adding a traceback.

Commit:

6856c0f7888f2c0aa822095145e82a0123ce1786

a3d18a843da1446ac5221387f4ba4b2cd1cf95aa

Diff:

Revision 2 (+74)

Show changes

djblets/webapi/resources/registry.py

Tool: PEP8 Style Checker
Processed Files:
    djblets/webapi/resources/registry.py



Tool: Pyflakes
Processed Files:
    djblets/webapi/resources/registry.py

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to release-0.9.x (b105111)