rubycritic 0.0.5 → 0.0.16

data/lib/rubycritic/{smell.rb → core/smell.rb}

@@ -1,5 +1,5 @@
 require "virtus"
-require "rubycritic/location"
+require "rubycritic/core/location"
 
 module Rubycritic
 
@@ -7,17 +7,14 @@ module Rubycritic
     include Virtus.model
 
     attribute :context
+    attribute :cost
     attribute :locations
     attribute :message
     attribute :score
-    attribute :type
     attribute :status
+    attribute :type
 
-    def pathnames
-      @pathnames ||= locations.map(&:pathname).uniq
-    end
-
-    def located_in?(other_location)
+    def at_location?(other_location)
       locations.any? { |location| location == other_location }
     end
 
@@ -26,14 +23,10 @@ module Rubycritic
     end
 
     def ==(other)
-      self.class == other.class && state == other.state
+      state == other.state
     end
     alias_method :eql?, :==
 
-    def <=>(other)
-      locations <=> other.locations
-    end
-
     def to_s
       "(#{type}) #{context} #{message}"
     end

data/lib/rubycritic/files_initializer.rb

@@ -0,0 +1,15 @@
+require "rubycritic/source_locator"
+require "rubycritic/core/analysed_file"
+
+module Rubycritic
+
+  module FilesInitializer
+    def self.init(paths)
+      source = SourceLocator.new(paths)
+      source.pathnames.map do |pathname|
+        AnalysedFile.new(:pathname => pathname, :smells => [], :duplication => 0)
+      end
+    end
+  end
+
+end

data/lib/rubycritic/orchestrator.rb

@@ -0,0 +1,23 @@
+require "rubycritic/source_control_systems/base"
+require "rubycritic/files_initializer"
+require "rubycritic/analysers_runner"
+require "rubycritic/revision_comparator"
+
+module Rubycritic
+
+  class Orchestrator
+    def initialize
+      @source_control_system = SourceControlSystem::Base.create
+    end
+
+    def critique(paths)
+      analysed_files = FilesInitializer.init(paths)
+      AnalysersRunner.new(analysed_files, @source_control_system).run
+      if @source_control_system.has_revision?
+        RevisionComparator.new(analysed_files, @source_control_system).set_statuses
+      end
+      analysed_files
+    end
+  end
+
+end

data/lib/rubycritic/report_generators/assets/javascripts/application.js

@@ -1,17 +1,39 @@
 prettyPrint();
 
-highlightLineFromFragment();
+emphasizeLineFromFragment();
 
-function highlightLineFromFragment() {
-    highlightLine(window.location.hash)
+function emphasizeLineFromFragment() {
+    emphasizeLine(window.location.hash)
 }
 
-$(".js-file-code").on("click", ".js-smell-location", highlightSmellyLine);
+$(".js-file-code").on("click", ".js-smell-location", emphasizeLineFromHref);
 
-function highlightSmellyLine() {
-    var lineId = "#" + this.href.split("#")[1];
-    $(".js-file-code li").removeClass("highlight");
-    highlightLine(lineId);
+function emphasizeLineFromHref(event) {
+    if (hrefTargetIsOnCurrentPage(this) && !event.ctrlKey) {
+        $(".js-file-code li").removeClass("highlight");
+        var lineId = "#" + this.href.split("#")[1];
+        emphasizeLine(lineId);
+        return false;
+    }
+}
+
+function hrefTargetIsOnCurrentPage(aTag) {
+    return (window.location.pathname === aTag.pathname);
+}
+
+function emphasizeLine(lineReference) {
+    scrollToTarget(lineReference);
+    highlightLine(lineReference);
+}
+
+function scrollToTarget(lineReference) {
+    $.scrollTo(lineReference, {
+        duration: 300,
+        easing: "linear",
+        onAfter: function() {
+            window.location.hash = lineReference;
+        }
+    });
 }
 
 function highlightLine(lineReference) {
@@ -23,3 +45,60 @@ $("#js-toggle-smells").on("click", toggleSmells);
 function toggleSmells() {
     $(".js-smells").toggle();
 }
+
+$("#js-index-table")
+    .tablesorter({          // Sort the table
+        sortList: [[0,0]]   // on the first column, in ascending order
+    })
+    .floatThead();          // Make table headers stick to the top when scrolling
+
+$(".js-timeago").timeago();
+
+$("#js-chart-container").highcharts({
+    chart: {
+        type: "scatter",
+        zoomType: "xy"
+    },
+    title: {
+        text: "Churn vs Complexity"
+    },
+    xAxis: {
+        title: {
+            enabled: true,
+            text: "Churn"
+        },
+        floor: 0,
+        startOnTick: true,
+        endOnTick: true
+    },
+    yAxis: {
+        title: {
+            text: "Complexity"
+        },
+        floor: 0,
+        startOnTick: true,
+        endOnTick: true
+    },
+    plotOptions: {
+        scatter: {
+            marker: {
+                radius: 5,
+                states: {
+                    hover: {
+                        enabled: true,
+                        lineColor: "rgb(100,100,100)"
+                    }
+                }
+            },
+            tooltip: {
+                headerFormat: "<b>{point.key}</b><br>",
+                pointFormat: "Committed {point.x} times, with Flog score of {point.y}"
+            }
+        }
+    },
+    series: [{
+        showInLegend: false,
+        color: "steelblue",
+        data: turbulenceData
+    }]
+});