Summary

Fix references to AvatarService.id

Review Request #8004 — Created Feb. 26, 2016 and submitted Feb. 26, 2016, 4:22 p.m.

Information

Owner

brennie

Repository

Review Board

Branch

release-2.6.x

Bugs

Depends On

Reviewers

Groups

reviewboard

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.
This patch also updates the user resource to indicate multiple URLs are

being returned, as well as fixing a missing size parameter.

Testing Done


Ran unit tests.

Issues

Description	From	Last Updated
The changes to this file don't seem to be part of the rest of the change? Did you intend to …	david	Feb. 26, 2016, 1:37 p.m.

Tool: Pyflakes
Processed Files:
    reviewboard/avatars/tests.py
    reviewboard/admin/forms.py
    reviewboard/webapi/resources/user.py
    reviewboard/avatars/registry.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/avatars/tests.py
    reviewboard/admin/forms.py
    reviewboard/webapi/resources/user.py
    reviewboard/avatars/registry.py

reviewboard/webapi/resources/user.py (Diff revision 1)

The issue has been dropped. Show all issues

The changes to this file don't seem to be part of the rest of the change? Did you intend to have this in here?

brennie Feb. 26, 2016, 1:37 p.m.
```
Oops forgot to mention this.
```

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.
+
+
This patch also updates the user resource to indicate multiple URLs are
+ being returned, as well as fixing a missing size parameter.

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to release-2.6.x (2bca118)

		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.
	+
	+	This patch also updates the user resource to indicate multiple URLs are
	+	being returned, as well as fixing a missing `size` parameter.