[Piuparts-commits] rev 632 - piatti/org/piuparts.debian.org/htdocs trunk

[Holger Levsen]

Author: holger
Date: 2010-01-09 14:23:41 +0000 (Sat, 09 Jan 2010)
New Revision: 632

Modified:
   piatti/org/piuparts.debian.org/htdocs/style.css
   trunk/piuparts-report.py
Log:
emphasize bugs to filed, not filed ones

Modified: piatti/org/piuparts.debian.org/htdocs/style.css
===================================================================
--- piatti/org/piuparts.debian.org/htdocs/style.css	2010-01-07 10:21:30 UTC (rev 631)
+++ piatti/org/piuparts.debian.org/htdocs/style.css	2010-01-09 14:23:41 UTC (rev 632)
@@ -169,15 +169,19 @@
  padding-right: 6px;
 }
 
-a.bugged {
+a.needs-bugging {
  background-color:#e0c0d0;
 }
 
-#bugged {
+#needs-bugging {
  background-color:#e0c0d0;
 }
 
+a.bugged {
+ background-color:#9090d0;
+}
 
+
 a:link {
  color: #0000FF;
  text-decoration: none;

Modified: trunk/piuparts-report.py
===================================================================
--- trunk/piuparts-report.py	2010-01-07 10:21:30 UTC (rev 631)
+++ trunk/piuparts-report.py	2010-01-09 14:23:41 UTC (rev 632)
@@ -587,10 +587,14 @@
     def find_links_to_logs(self, package_name, dirs, logs_by_dir):
         links = []
         for dir in dirs:
+          if dir == "fail":
+            style = " class=\"needs-bugging\""
+          else
+            style = ""
           for basename in logs_by_dir[dir]:
             if basename.startswith(package_name+"_") and basename.endswith(".log"):
               package, version = basename[:-len(".log")].split("_")
-              links.append("<a href=\"/%s\">%s</a>" % (os.path.join(self._config.section, dir, basename),html_protect(version)))
+              links.append("<a href=\"/%s\"%s>%s</a>" % (os.path.join(self._config.section, dir, basename),style,html_protect(version)))
         return links
 
     def link_to_maintainer_summary(self, maintainer):
@@ -872,11 +876,11 @@
               if count_bugged != 0 or count_failed != 0:
                 substats = ": "
               if count_bugged != 0:
-                substats += "<span id=\"bugged\">%s bugged</span>" % count_bugged
+                substats += "%s bugged" % count_bugged
               if count_bugged != 0 and count_failed != 0:
                 substats += ", "
               if count_failed != 0:
-                substats += "%s failed" % count_failed 
+                substats += "<span id=\"needs-bugging\">%s failed</span>" % count_failed
             link += "<li><a href=%s>%s</a>%s</li>" % (
                                                                        template[:-len(".tpl")]+".html", 
                                                                        linktarget_by_template[template],