Merge lp:~primes2h/ubuntu-qa-website/bugs_733364_733369 into lp:~ubuntu-qa-website-devel/ubuntu-qa-website/trunk

Proposed by Sergio Zanchetta
Status: Merged
Merged at revision: 155
Proposed branch: lp:~primes2h/ubuntu-qa-website/bugs_733364_733369
Merge into: lp:~ubuntu-qa-website-devel/ubuntu-qa-website/trunk
Diff against target: 49 lines (+6/-5)
1 file modified
code/modules/qatracker/qatracker.display.php (+6/-5)
To merge this branch: bzr merge lp:~primes2h/ubuntu-qa-website/bugs_733364_733369
Reviewer Review Type Date Requested Status
Ubuntu QA Website Developers Pending
Review via email: mp+53111@code.launchpad.net

Description of the change

Laptop tracker:
 * Made -Not tested- linkable to allow testing. (LP:#733364)
 * Linked testcase icon to testcase page. (LP:#733369)
 * Added two whitespaces before laptop name in summary table, adjusted comment formatting in balloons.

Database:
In qawebsite_user_setting table, site specific descriptions needs to be uniformed and commands highlighted.

Run sudo dmidecode -s system-manufacturer → Run "<b>sudo dmidecode -s system-manufacturer</b><br \>and capitalize first letter of the output only (e.g. TOSHIBA → Toshiba)"

Run sudo dmidecode -s system-product-name → Run "<b>sudo dmidecode -s system-product-name</b>"

Run `lspci -v` → Run "<b>lspci -v</b>"

Run `lsusb` → Run "<b>lsusb</b>" without any external device attached

`lshw -short` → Run "<b>lshw -short</b>"

P.S.:" " could also be ` `

To post a comment you must log in.
156. By Sergio Zanchetta

Added two whitespaces before laptop name in summary table, adjusted comment formatting in balloons.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'code/modules/qatracker/qatracker.display.php'
--- code/modules/qatracker/qatracker.display.php 2011-03-09 14:32:59 +0000
+++ code/modules/qatracker/qatracker.display.php 2011-03-15 14:33:34 +0000
@@ -923,7 +923,7 @@
923 WHERE siteid='".$site->id."' AND title LIKE '".$release."%' AND productid='".$productID."' 923 WHERE siteid='".$site->id."' AND title LIKE '".$release."%' AND productid='".$productID."'
924 ORDER BY qatracker_milestone.id");924 ORDER BY qatracker_milestone.id");
925925
926 $testcases=db_query("SELECT id,title FROM qatracker_testcase WHERE productid='".$productID."'");926 $testcases=db_query("SELECT id,title,link FROM qatracker_testcase WHERE productid='".$productID."'");
927 $product = db_result(db_query("SELECT title from qatracker_product WHERE id='".$productID."'"));927 $product = db_result(db_query("SELECT title from qatracker_product WHERE id='".$productID."'"));
928928
929 $milestones=array();929 $milestones=array();
@@ -972,7 +972,7 @@
972 $content.="</tr>";972 $content.="</tr>";
973 }973 }
974 $content.=" <tr class=\"block".$productID."".$countv."\" style=\"display:none;\">974 $content.=" <tr class=\"block".$productID."".$countv."\" style=\"display:none;\">
975 <td style=\"width=230px;\">&nbsp; <img style=\"margin-bottom:0px;\" src=\"$base_url/modules/qatracker/images/test.png\" alt=\"spacer\" /> ".$title[1]."</td>";975 <td style=\"width=230px;\">&nbsp; <a rel=\"external\" href=\"".$testcase->link."\"><img style=\"margin-bottom:-3px;\" src=\"$base_url/modules/qatracker/images/test.png\" alt=\"spacer\" /></a> ".$title[1]."</td>";
976 foreach($milestones as $build => $milestone) {976 foreach($milestones as $build => $milestone) {
977 $results=db_fetch_object(db_query("SELECT result,comment FROM qatracker_result WHERE reporterid='".$reporterID."' AND buildid='".$build."' AND testcaseid='".$testcase->id."'"));977 $results=db_fetch_object(db_query("SELECT result,comment FROM qatracker_result WHERE reporterid='".$reporterID."' AND buildid='".$build."' AND testcaseid='".$testcase->id."'"));
978 if ($results->result != null) {978 if ($results->result != null) {
@@ -980,7 +980,7 @@
980 if ($results->comment != null) {980 if ($results->comment != null) {
981 $content.="<b style=\"margin-right:-32px;font-size:10px;\">?</b>981 $content.="<b style=\"margin-right:-32px;font-size:10px;\">?</b>
982 <div class=\"balloonleft\" style=\"margin-bottom:0px;\"><div>982 <div class=\"balloonleft\" style=\"margin-bottom:0px;\"><div>
983 <b>Comment: <br \></b>".$results->comment."</div>";983 <b>Comment: <br \></b>".nl2br($results->comment)."</div>";
984 }984 }
985 switch($results->result) {985 switch($results->result) {
986 case "0":986 case "0":
@@ -995,7 +995,8 @@
995 $content.="&nbsp;".qatracker_getbugslist($build, $testcase->id, 0)."<br \></td>";995 $content.="&nbsp;".qatracker_getbugslist($build, $testcase->id, 0)."<br \></td>";
996 }996 }
997 else997 else
998 $content.="<td cellpadding=\"10\" style=\"text-align:center;font-size:10px;\">Not <br \> tested</td>";998 $content.="<td cellpadding=\"10\" style=\"text-align:center;font-size:10px;\">
999 <a style=\"color:#000000\" rel=\"external\" href=\"$base_url/qatracker/result/".$build."/".$testcase->id."\">Not <br \> tested</a></td>";
999 }1000 }
1000 $content.=" </tr>";1001 $content.=" </tr>";
1001 }1002 }
@@ -1064,7 +1065,7 @@
1064 $content.="<tr><td colspan=\"3\" style=\"background-color:#e6e6e6;\"><b>".$laptopmake."</b></td></tr>";1065 $content.="<tr><td colspan=\"3\" style=\"background-color:#e6e6e6;\"><b>".$laptopmake."</b></td></tr>";
1065 }1066 }
1066 $content.=" <tr>1067 $content.=" <tr>
1067 <td style=\"width:20%;\">&nbsp;".$laptopname."</td>1068 <td style=\"width:20%;\">&nbsp;&nbsp;&nbsp;".$laptopname."</td>
1068 <td style=\"width:30%;\">&nbsp;".$reporter."&nbsp;→&nbsp;<a rel=\"external\" href=\"https://launchpad.net/~".$lpusername[$reporter]."\">".$lpusername[$reporter]."</a></td>";1069 <td style=\"width:30%;\">&nbsp;".$reporter."&nbsp;→&nbsp;<a rel=\"external\" href=\"https://launchpad.net/~".$lpusername[$reporter]."\">".$lpusername[$reporter]."</a></td>";
10691070
1070 $content.=" <td><table>";1071 $content.=" <td><table>";

Subscribers

People subscribed via source and target branches

to status/vote changes: