lib/build-tool/model/module.rb in build-tool-0.6.7 vs lib/build-tool/model/module.rb in build-tool-0.6.8

@@ -84,10 +84,17 @@
             else
                 return active
             end
         end
 
+        def dirty?
+            if vcs
+                return vcs.dirty?
+            end
+            return false
+        end
+
         # not inherited
         def build_directory
             build_prefix_required.join("bld", local_path)
         end
 
@@ -263,16 +270,21 @@
             return BuildTool::History::CommandLog.last_by_module( name )
         end
 
         # Returns a current state in string format
         def state
-            return 'UNKNOWN' if lastlog.empty?
-            lastlog[0].module_logs.where( :module => name ).each do |e|
-                if e.state != History::ModuleLog::FINISHED_SUCCESSFUL
-                    return "#{e.state_str} (#{e.event})"
-                end
+            log = lastlog
+            return 'UNKNOWN' if log.empty?
+
+            e = log[0].module_logs.reverse_order.where( :module => name ).first
+            if e.nil?
+                return "UNKNOWN"
             end
-            return History::ModuleLog::state_str( History::ModuleLog::FINISHED_SUCCESSFUL )
+            if e.state != History::ModuleLog::FINISHED_SUCCESSFUL
+                return "#{e.started_at.strftime("%x %X")}: #{e.state_str} (#{e.event})"
+            else
+                return "#{e.started_at.strftime("%x %X")}: #{e.state_str}"
+            end
         end
 
         # Return the current state as one char.
         def state_char
             return '?' if lastlog.empty?