tumblargh 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0e50e3d39c5dc3fb912cfcaf48113581fb644456
-  data.tar.gz: 66980f7b07a44cb19b191f86f3c1ff10fd6a31ac
+  metadata.gz: 2797b0ef63ebf34cc39bd5287f6a6a62f4fd2123
+  data.tar.gz: a1b6a5bb0ebeea850dd41b038d8c2f87230d8a9c
 SHA512:
-  metadata.gz: a37b11aa4bc7d25724f36039cbf566e4f46362be27e13d655fadd4a039d69e9549bbc774150e034142302ccccd35db5b2cb4b6a7be812d04dab621fd0da95497
-  data.tar.gz: c318b3856836d9f07d7812624c877555963153585324a924f982dc84bf727b91c1a4ddd18c7454f85824efffa8f5258b412b35beaf55c9d14cc69796ff2e6803
+  metadata.gz: 25e5eaac7e0c20219cd221b2549ce9e7ad34e2234f69d27df36e8b67d5ae473aa2c6839afe690b56ed4d8b9d2e7187837e4a86d2ae33f52eb3fbf2b39a3f45f8
+  data.tar.gz: 87e6edc18b48d1d240ba5b8431a6d445a31e2a31cbc082c6126ffe0e93aff7d1e7ac7b404a6e9e68796ea17f11f4aac2e3f91445629a36f6dbfdf3bfc9d14eb5

data/.travis.yml

@@ -0,0 +1,7 @@
+language: ruby
+rvm:
+  - 2.0.0
+  - 1.9.3
+
+notifications:
+  email: false

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 0.2.2
+
+- Don't raise an exception on missing blocks, merely log a warning
+- Implement `{TagsAsClasses}` tag
+
 ## 0.2.1
 
 - Block arguments, such as `{block:JumpPagination length="5"}` are now supported

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    tumblargh (0.2.0)
+    tumblargh (0.2.1)
       activesupport (>= 3.1)
       api_cache
       nokogiri
@@ -35,7 +35,7 @@ GEM
       multi_json (~> 1.0)
       simplecov-html (~> 0.5.3)
     simplecov-html (0.5.3)
-    treetop (1.4.14)
+    treetop (1.4.15)
       polyglot
       polyglot (>= 0.3.1)
 

data/README.md

@@ -1,4 +1,7 @@
 # Tumblargh
+
+[![Build Status](https://travis-ci.org/jasonwebster/tumblargh.png?branch=master)](https://travis-ci.org/jasonwebster/tumblargh)
+
 ## Groan-less Tumblr theme development
 
 ### What is this thing, and why should I care?

data/lib/tumblargh/renderer.rb

@@ -29,13 +29,16 @@ module Tumblargh
           args << n_post[1].to_i
         end
 
-        base = "Blocks::#{block_name}"
+        base = "Blocks::#{ block_name }"
       end
 
-      klass_name = "Tumblargh::Renderer::#{base}"
-      klass = klass_name.constantize
-
-      klass.new(node, context, options, *args)
+      klass_name = "Tumblargh::Renderer::#{ base }"
+      begin
+        klass = klass_name.constantize
+        klass.new(node, context, options, *args)
+      rescue NameError => e
+        puts "WARNING: Unsupported block `#{ klass_name }`"
+      end
     end
 
   end

data/lib/tumblargh/renderer/base.rb

@@ -55,7 +55,6 @@ module Tumblargh
       alias_method :to_s, :render
 
       def method_missing(method, *arguments)
-        # puts "mm #{method} - #{self.class.name} -- context is: #{context.class.name}"
         raise "Can't find anything to do with '#{method}'" if context.nil?
         context.send(method, *arguments)
       end

data/lib/tumblargh/renderer/blocks/base.rb

@@ -9,7 +9,6 @@ module Tumblargh
           alias_method :should_render_unless_empty, :should_render_if=
         end
 
-
         def should_render?
           if defined?(@should_render_if)
             val = send(@should_render_if)
@@ -25,10 +24,11 @@ module Tumblargh
           _, type, options, *nodes = node
 
           res = nodes.map do |n|
-            Renderer.factory(n, self, options).render
+            renderer = Renderer.factory(n, self, options)
+            renderer.render unless renderer.nil?
           end
 
-          " #{res.join('')} "
+          " #{ res.join('') } "
         end
       end
     end

data/lib/tumblargh/renderer/blocks/posts.rb

@@ -3,7 +3,7 @@ module Tumblargh
     module Blocks
       # Posts Loop
       #
-      # {block:Posts} is executed once for each post. Some post related tags can 
+      # {block:Posts} is executed once for each post. Some post related tags can
       # exist outside of a `type` block, such as {Title} or {Permalink}, so
       # they should be defined here
       class Posts < Base
@@ -31,6 +31,22 @@ module Tumblargh
           "/reblog/#{context.reblog_key}"
         end
 
+        # An HTML class-attribute friendly list of the post's tags.
+        # Example: "humor office new_york_city"
+        #
+        # By default, an HTML friendly version of the source domain of imported
+        # posts will be included. This may not behave as expected with feeds
+        # like Del.icio.us that send their content URLs as their permalinks.
+        # Example: "twitter_com", "digg_com", etc.
+        #
+        # The class-attribute "reblog" will be included automatically if the
+        # post was reblogged from another post.
+        def tags_as_classes
+          context.tags.map do |tag|
+            tag.name.titlecase.gsub(/\s+/, '').underscore.downcase
+          end.join " "
+        end
+
         def render
           if context.is_a? Resource::Post
             super

data/lib/tumblargh/renderer/document.rb

@@ -55,7 +55,8 @@ module Tumblargh
 
       def render
         node.map do |n|
-          Renderer.factory(n, self).render
+          renderer = Renderer.factory(n, self)
+          renderer.render unless renderer.nil?
         end.flatten.join('')
       end
     end

data/lib/tumblargh/version.rb

@@ -1,3 +1,3 @@
 module Tumblargh
-  VERSION = '0.2.1'.freeze
+  VERSION = '0.2.2'.freeze
 end

data/spec/renderer/blocks/posts_spec.rb

@@ -14,4 +14,13 @@ describe Tumblargh::Renderer::Blocks::Posts do
     result.count("!").should eql @json[:posts].size
   end
 
+  describe "{TagsAsClasses}" do
+    let(:post) { Tumblargh::Resource::Post.new({ tags: ["TROLOLOL", "Herp derp"] }, nil) }
+
+    subject { Tumblargh::Renderer::Blocks::Posts.new(nil, post) }
+
+    it "should give the tags as class names" do
+      subject.tags_as_classes.should eql "trololol herp_derp"
+    end
+  end
 end

data/spec/renderer/blocks_spec.rb

@@ -3,13 +3,22 @@ require 'spec_helper'
 
 describe Tumblargh::Renderer::Blocks do
 
-  before do
-    @parser = Tumblargh::Parser.new
+  def capture_stdout(&block)
+    original_stdout = $stdout
+    $stdout = fake = StringIO.new
+    begin
+      yield
+    ensure
+      $stdout = original_stdout
+    end
+    fake.string
   end
 
+  let(:parser) { Tumblargh::Parser.new }
+
   describe "{block:PermalinkPage}" do
     before do
-      @parser.html = <<-eos
+      parser.html = <<-eos
         {block:PermalinkPage}
         ERMAGERD PERMERLINK
         {/block:PermalinkPage}
@@ -17,23 +26,21 @@ describe Tumblargh::Renderer::Blocks do
     end
 
     it "should render its content when it is a permalink page" do
-      @document = Tumblargh::Renderer::Document.new(@parser.tree, nil, :permalink => true)
-      @output = @document.render
-      @output.should match(/ERMAGERD PERMERLINK/)
+      document = Tumblargh::Renderer::Document.new(parser.tree, nil, :permalink => true)
+      output = document.render
+      output.should match(/ERMAGERD PERMERLINK/)
     end
 
     it "should NOT render its content when it is a permalink page" do
-      @document = Tumblargh::Renderer::Document.new(@parser.tree, nil, :permalink => false)
-      @output = @document.render
-      @output.should_not match(/ERMAGERD PERMERLINK/)
+      document = Tumblargh::Renderer::Document.new(parser.tree, nil, :permalink => false)
+      output = document.render
+      output.should_not match(/ERMAGERD PERMERLINK/)
     end
-
   end
 
   describe "boolean blocks" do
-
     before do
-      @parser.html = <<-eos
+      parser.html = <<-eos
       {block:IfSomethingOnByDefault}
       PASS_ON
       {/block:IfSomethingOnByDefault}
@@ -50,29 +57,51 @@ describe Tumblargh::Renderer::Blocks do
       PASS_INVERSE_OFF
       {/block:IfNotSomethingOffByDefault}
       eos
+    end
 
-      options = {
+    let(:options) do
+      {
         "if" => {
           "somethingonbydefault" => true,
           "somethingoffbydefault" => false
         }
       }
-
-      @document = Tumblargh::Renderer::Document.new(@parser.tree, nil, options)
-      @output = @document.render
     end
 
+    let(:output) { Tumblargh::Renderer::Document.new(parser.tree, nil, options).render }
+
     it "should respect the default settings" do
-      @output.should match(/PASS_ON/)
-      @output.should_not match(/FAIL_OFF/)
+      output.should match(/PASS_ON/)
+      output.should_not match(/FAIL_OFF/)
     end
 
 
     it "inverse (IfNot) blocks should work" do
-      @output.should_not match(/FAIL_INVERSE_ON/)
-      @output.should match(/PASS_INVERSE_OFF/)
+      output.should_not match(/FAIL_INVERSE_ON/)
+      output.should match(/PASS_INVERSE_OFF/)
     end
-
   end
 
+  describe "using unsupported blocks" do
+    before do
+      parser.html = <<-eos
+      {block:ThisIsDefinitelyNotABlock}{/block:ThisIsDefinitelyNotABlock}
+      eos
+    end
+
+    it "should not raise an error" do
+      expect {
+        Tumblargh::Renderer::Document.new(parser.tree, nil, {}).render
+      }.not_to raise_error
+    end
+
+    it "should output a warning" do
+      out = capture_stdout {
+        Tumblargh::Renderer::Document.new(parser.tree, nil, {}).render
+      }
+
+      out.should =~ /WARNING: Unsupported block `Tumblargh::Renderer::Blocks::ThisIsDefinitelyNotABlock`/
+    end
+
+  end
 end

data/spec/resource/post_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Tumblargh::Resource::Post do
-  
+
   before do
     json_path = File.join(FIXTURE_PATH, "data", "staff.tumblr.com-2012-05-06", "posts.json")
     @json = ActiveSupport::JSON.decode(open(json_path).read)["response"]
@@ -33,6 +33,4 @@ describe Tumblargh::Resource::Post do
     end
   end
 
-
-
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tumblargh
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Jason Webster
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-09 00:00:00.000000000 Z
+date: 2013-10-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -132,6 +132,7 @@ extra_rdoc_files: []
 files:
 - .gitignore
 - .rspec
+- .travis.yml
 - CHANGELOG.md
 - Gemfile
 - Gemfile.lock