Merge lp:~pfalcon/linaro-android-frontend/fix-config-lookup into lp:linaro-android-frontend

Proposed by Paul Sokolovsky
Status: Merged
Merged at revision: 231
Proposed branch: lp:~pfalcon/linaro-android-frontend/fix-config-lookup
Merge into: lp:linaro-android-frontend
Diff against target: 28 lines (+10/-8)
1 file modified
static/buildDetails.js (+10/-8)
To merge this branch: bzr merge lp:~pfalcon/linaro-android-frontend/fix-config-lookup
Reviewer Review Type Date Requested Status
James Westby (community) Approve
Michael Hudson-Doyle Pending
Review via email: mp+72407@code.launchpad.net

Description of the change

This fixes "???" displayed for config for some jobs in https://android-build.linaro.org frontend.

To post a comment you must log in.
Revision history for this message
James Westby (james-w) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'static/buildDetails.js'
--- static/buildDetails.js 2011-06-21 17:38:09 +0000
+++ static/buildDetails.js 2011-08-22 11:12:25 +0000
@@ -31,14 +31,16 @@
31 return results.result || "RUNNING";31 return results.result || "RUNNING";
32 },32 },
33 configuration: function (results) {33 configuration: function (results) {
34 if (results.actions[0].parameters) {34 for (var i = 0; i < results.actions.length; i++) {
35 var params = results.actions[0].parameters;35 if (results.actions[i].parameters) {
36 for (var i = 0; i < params.length; i++) {36 var params = results.actions[i].parameters;
37 if (params[i].name == "CONFIG") {37 for (var j = 0; j < params.length; j++) {
38 try {38 if (params[j].name == "CONFIG") {
39 return Y.Base64.decode(params[i].value);39 try {
40 } catch (error) {40 return Y.Base64.decode(params[j].value);
41 return 'Not Base64 encoded?';41 } catch (error) {
42 return 'Not Base64 encoded?';
43 }
42 }44 }
43 }45 }
44 }46 }

Subscribers

People subscribed via source and target branches