Fixes bug 806994 - Added functional tests where missing in the middleware. #1045

Merged
merged 1 commit into from Jan 17, 2013

Projects

None yet

2 participants

@adngdb
Member
adngdb commented Jan 11, 2013
@peterbe peterbe was assigned Jan 16, 2013
@peterbe peterbe commented on an outdated diff Jan 16, 2013
socorro/external/postgresql/report.py
@@ -74,7 +76,7 @@ def get_list(self, **kwargs):
# Changing the OS ids to OS names
for i, elem in enumerate(params["os"]):
for platform in context.platforms:
- if platform["id"] == elem:
+ if platform["id"][0:3] == elem[0:3]:
@peterbe
peterbe Jan 16, 2013 Contributor

Write [:3] to get the first 3 characters.

@peterbe
Contributor
peterbe commented Jan 16, 2013

r+ well done!

@adngdb
Member
adngdb commented Jan 17, 2013

Nit addressed, let's merge!

@adngdb adngdb merged commit 21a8fba into mozilla:master Jan 17, 2013

1 check passed

default Jenkins build 'socorro-github' #393 has succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment