Summary

Fix references to AvatarService.id

Review Request #8003 — Created Feb. 26, 2016 and submitted Feb. 26, 2016, 9:39 p.m.

Information

Owner

brennie

Repository

Djblets

Branch

release-0.10.x

Bugs

Depends On

Commit

1022d99...

Reviewers

Groups

djblets

People

Description

In review, the id attribute of avatar services was changed to

avatar_service_id. This patch corrects all instances that were missed,

as well as fixing some unit tests that were missing an HTML class name

in the expected output.

Testing Done


Ran unit tests.
Tested this patch with Review Board.

Issues

Description	From	Last Updated
Col: 80 E501 line too long (84 > 79 characters)	reviewbot	Feb. 26, 2016, 5:08 p.m.
Col: 80 E501 line too long (80 > 79 characters)	reviewbot	Feb. 26, 2016, 5:08 p.m.
Col: 80 E501 line too long (91 > 79 characters)	reviewbot	Feb. 26, 2016, 5:08 p.m.

Tool: Pyflakes
Processed Files:
    djblets/avatars/tests.py
    djblets/avatars/registry.py
    djblets/avatars/services/base.py



Tool: PEP8 Style Checker
Processed Files:
    djblets/avatars/tests.py
    djblets/avatars/registry.py
    djblets/avatars/services/base.py

djblets/avatars/registry.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (84 > 79 characters)
```
djblets/avatars/registry.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (80 > 79 characters)
```
djblets/avatars/registry.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (91 > 79 characters)
```

Change Summary:

PEP8

Testing Done:

		Ran unit tests.
~		Tested this with Review Board.
	~	Tested this patch with Review Board.

Diff:

Revision 2 (+58 -40)

Show changes

	djblets/avatars/registry.py
	djblets/avatars/tests.py
	djblets/avatars/services/base.py

Tool: Pyflakes
Processed Files:
    djblets/avatars/tests.py
    djblets/avatars/registry.py
    djblets/avatars/services/base.py



Tool: PEP8 Style Checker
Processed Files:
    djblets/avatars/tests.py
    djblets/avatars/registry.py
    djblets/avatars/services/base.py

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to release-0.10.x (e4aef44)

djblets/avatars/registry.py (Diff revision 1)

djblets/avatars/registry.py (Diff revision 1)

djblets/avatars/registry.py (Diff revision 1)