code_poetry-html 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a59640d523afcdcd22294bf2fff32d8f957ec747
-  data.tar.gz: c3cdeedd0d6fdb0aafc9e556f20c6d9f0a6ec540
+  metadata.gz: aca1ce4858a5fba3552589e9842ff2a1494308f0
+  data.tar.gz: f2f5cfc2927b4606a953d67cde8b52b4e0e0d4b8
 SHA512:
-  metadata.gz: a14d366c0dc71769896451bfb614cd6c84d634a8b11ded62a529149229a8d46e258181f3877e75ba7efc4b3fbaec13982a477cd5616c6a046d8538a2edfc863b
-  data.tar.gz: bca34bc40ddfce47b8021d230669a53aabc121be6dc2aa6c5c07dd435738db1a6f48a06e2237f374aece1bb2418fa07641f2a8cd490df2d004d5b0ffc4955934
+  metadata.gz: 99d4f59f0bf00598438c171e5c6fe4de2e1c02b246198007714482ff8c7d848b369510f513820becd5df72eaf974cd763a4049d439c5a3b3846f9223eb1aa37f
+  data.tar.gz: 0fa67896841509c098b1ce35ec6963ca163011260b2765b62b7a506623ab71fa579f77a356ec1d5b4d586b97ec0d9b021aa422c472e29eeea8c4baef1425b091

data/README.md

@@ -4,9 +4,9 @@
 
 It generates a clean and simple HTML report for [CodePoetry][cp].
 
-[rgb]: https://badge.fury.io/rb/code_poetry-html.png
+[rgb]: https://codeclimate.com/github/coding-chimp/code_poetry-html/badges/gpa.svg
 [rgl]: http://badge.fury.io/rb/code_poetry-html
-[ccb]: https://codeclimate.com/github/coding-chimp/code_poetry-html.png
+[ccb]: https://codeclimate.com/github/coding-chimp/code_poetry-html/badges/gpa.svg
 [ccl]: https://codeclimate.com/github/coding-chimp/code_poetry-html
 
 [cp]: https://github.com/coding-chimp/code_poetry

data/assets/stylesheets/screen.css.scss

@@ -1,5 +1,8 @@
+$default-font: 'Lato', sans-serif;
+$code-font: Consolas, 'Bitstream Vera Sans Mono', Monaco, Courier, monospace;
+
 body {
-  font-family: 'Lato', sans-serif;
+  font-family: $default-font;
   padding: 12px 24px;
   color: #777;
 }
@@ -20,7 +23,7 @@ table { width: 100% }
 }
 
 .header h1 {
-  font-family: 'Lato', sans-serif;
+  font-family: $default-font;
   margin: 0.2em 0;
   font-size: 4em;
   font-weight: 300;
@@ -79,9 +82,9 @@ th {
     padding: 0;
     white-space: normal;
     color: #000;
-    font-family: Consolas, 'Bitstream Vera Sans Mono', Monaco, Courier, monospace;
+    font-family: $code-font;
     p {
-      font-family: 'Lato', sans-serif;
+      font-family: $default-font;
       color: white;
       margin: 0px;
       padding: 10px 0 10px 15px;
@@ -89,7 +92,7 @@ th {
   }
   code {
     color: #000;
-    font-family: Consolas, 'Bitstream Vera Sans Mono', Monaco, Courier, monospace;
+    font-family: $code-font;
   }
   pre {
     background-color: #333;
@@ -112,8 +115,16 @@ th {
     li:first-child { padding-top: 4px }
     li:last-child { padding-bottom: 4px }
   }
+  .smell-description {
+    margin-left: 45px;
+    padding: 5px 15px;
+    border-left: 7px solid #ee7b10;
+    background-color: #ee7b10;
+    color: white;
+    font-family: $default-font;
+  }
   .smelly {
-    border-left:7px solid #ee7b10
+    border-left: 7px solid #ee7b10
   }
   .smelly:nth-child(even), .smelly:nth-child(even) code {
     background-color: #f7e9d0

data/lib/code_poetry-html/version.rb

@@ -1,7 +1,7 @@
 module CodePoetry
   module Formatter
     class HTMLFormatter
-      VERSION = "0.2.0"
+      VERSION = "0.3.0"
     end
   end
 end

data/lib/code_poetry-html.rb

@@ -47,7 +47,7 @@ private
   end
 
   def id(stat)
-    Digest::SHA1.hexdigest(stat.file)
+    Digest::SHA1.hexdigest(stat.absolute_path)
   end
 
   def timeago(time)
@@ -65,6 +65,21 @@ private
       'smelly' if method.smelly? || method.duplicated?
     end
   end
+
+  def first_line_of_smell?(stat, line_number)
+    line_status(stat, line_number - 1).nil? &&
+      line_status(stat, line_number) == 'smelly'
+  end
+
+  def smell_title(stat, line_number)
+    method = stat.get_method_at_line(line_number)
+
+    if method.smelly?
+      'Complex Method'
+    else
+      'Duplication'
+    end
+  end
 end
 
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__)))

data/public/application.css

@@ -419,7 +419,7 @@ pre .tex .formula {
 	}
 }
 body {
-  font-family: 'Lato', sans-serif;
+  font-family: "Lato", sans-serif;
   padding: 12px 24px;
   color: #777; }
 
@@ -438,7 +438,7 @@ table {
   text-align: center; }
 
 .header h1 {
-  font-family: 'Lato', sans-serif;
+  font-family: "Lato", sans-serif;
   margin: 0.2em 0;
   font-size: 4em;
   font-weight: 300;
@@ -475,7 +475,7 @@ th {
   list-style: none;
   margin-top: 10px; }
 .source_table .smell-view .smell {
-  color: black; }
+  color: #000; }
 .source_table .smell-view .smell-name {
   font-size: 18px; }
 .source_table .smell-view .smell-complexity {
@@ -487,15 +487,15 @@ th {
   padding: 0;
   white-space: normal;
   color: #000;
-  font-family: Consolas, 'Bitstream Vera Sans Mono', Monaco, Courier, monospace; }
+  font-family: Consolas, "Bitstream Vera Sans Mono", Monaco, Courier, monospace; }
   .source_table pre p {
-    font-family: 'Lato', sans-serif;
+    font-family: "Lato", sans-serif;
     color: white;
     margin: 0px;
     padding: 10px 0 10px 15px; }
 .source_table code {
   color: #000;
-  font-family: Consolas, 'Bitstream Vera Sans Mono', Monaco, Courier, monospace; }
+  font-family: Consolas, "Bitstream Vera Sans Mono", Monaco, Courier, monospace; }
 .source_table pre {
   background-color: #333; }
   .source_table pre ol {
@@ -515,6 +515,13 @@ th {
     padding-top: 4px; }
   .source_table pre li:last-child {
     padding-bottom: 4px; }
+.source_table .smell-title {
+  margin-left: 45px;
+  padding: 5px 15px;
+  border-left: 7px solid #ee7b10;
+  background-color: #ee7b10;
+  color: white;
+  font-family: "Lato", sans-serif; }
 .source_table .smelly {
   border-left: 7px solid #ee7b10; }
 .source_table .smelly:nth-child(even), .source_table .smelly:nth-child(even) code {

data/views/source_file.erb

@@ -24,10 +24,16 @@
 
   <div class="source-view" style="display: none;">
     <pre>
-      <p><%= stat.file %></p>
+      <p><%= stat.relative_path %></p>
       <ol>
         <% stat.lines.each do |number, line| %>
-          <li class="<%= line_status(stat, number) %>" data-linenumber="<%= number %>">
+          <% if first_line_of_smell?(stat, number) %>
+      </ol>
+      <div class="smell-description"><%= smell_title(stat, number) %></div>
+      <ol start="<%= number %>">
+          <% end %>
+
+          <li class="<%= line_status(stat, number) %>">
             <code class="ruby"><%= CGI.escapeHTML(line.chomp) %></code>
           </li>
         <% end %>

metadata

@@ -1,69 +1,69 @@
 --- !ruby/object:Gem::Specification
 name: code_poetry-html
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Bastian Bartmann
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-22 00:00:00.000000000 Z
+date: 2015-02-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: sprockets
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: sass
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Default HTML formatter for CodePoetry
@@ -73,7 +73,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
+- ".gitignore"
 - Gemfile
 - LICENSE
 - README.md
@@ -118,19 +118,18 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.3
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Default HTML formatter for CodePoetry
 test_files: []
-has_rdoc: