paloma 4.0.0 → 4.1.0

data/Changelog.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## Version 4.1.0
+* Support for Turbolinks.
+* `Paloma.executeHook()` to manually run the hook from the DOM.
+* Paloma hook will be appended using all `render` calls, except for calls that has the following keys `[:json, :js, :xml, :file]`.
+* Restore `Paloma.engine.start()` to start processing queued request.
+
+
 ## Version 4.0.0
 * https://github.com/kbparagua/paloma/issues/26 - Paloma requests are not saved on `session`.
 * https://github.com/kbparagua/paloma/issues/26 - Chaining with redirect is removed.

data/README.md

@@ -276,12 +276,43 @@ class UsersController < ApplicationController
 end
 ```
 
+
+## Turbolinks Support
+
+As of version `4.1.0`, Paloma is compatible with Turbolinks without additional setup.
+
+### Execute Paloma when user hits `Back` or `Forward` button.
+
+Paloma executes page-specific javascript by adding a `<script>` tag to the response body. Turbolinks, by default, executes any inline javascript in the response body when you visit a page, so the `<script>` tag appended by Paloma will automatically be executed. However, when Turbolinks restores a page from cache (*this happens when a user hits `Back` or `Forward` button in his browser*) any **inline javascript will not be executed** anymore. This is the intentional behavior of Turbolinks, and it is not a bug. If you want to execute Paloma again when Turbolinks restores a page, do something like this:
+
+```js
+$(document).on('page:restore', function(){
+  // Manually evaluates the appended script tag.
+  Paloma.executeHook();
+});
+```
+
 ## Gotchas
 
-* Paloma  will not execute if the response is `js`, `json`, `xml` or any other format except `html`.
+* Paloma  will execute on all `render` calls, except for calls with the following formats: `js`, `json`, `xml`, and `file`.
 
-For example: `render "something.js.erb"`
+   Example:
+   
+   ```ruby
+   render :json => {:x => 1}  # Paloma will not execute`
+   render :partial => '/path/to/partial'  # Paloma will execute
+   ```
 
+* It will cause conflicts if you have a controller and a module that has the same name.
+
+   Example:
+   ```js
+   var AdminController = Paloma.controller('Admin');
+   
+   // This will override the AdminController and replace it
+   // with a module named 'Admin'.
+   var UsersController = Paloma.controller('Admin/Users');
+   ```
 
 ## Where to put code?
 

data/app/views/paloma/_hook.html.erb

@@ -3,7 +3,6 @@
 <div class="js-paloma-hook" data-id="<%= id %>">
   <script type="text/javascript">
     (function(){
-
       // Do not continue if Paloma not found.
       if (window['Paloma'] === undefined && window['console'] !== undefined){
         console.warn("Paloma not found. Require it in your application.js.");
@@ -17,13 +16,12 @@
 
       var request = <%= request.to_json.html_safe %>;
 
-      $(document).ready(function(){
-        Paloma.engine.request(
-          request['resource'],
-          request['action'],
-          request['params']);
-      });
+      Paloma.engine.setRequest(
+        request['resource'],
+        request['action'],
+        request['params']);
 
+      $(document).ready(function(){ Paloma.engine.start(); });
     })();
   </script>
 </div>

data/lib/paloma/action_controller_extension.rb

@@ -12,7 +12,7 @@ module Paloma
         prepend_view_path "#{Paloma.root}/app/views/"
 
         before_filter :track_paloma_request
-        after_filter :append_paloma_hook, :if => :html_is_rendered?
+        after_filter :append_paloma_hook, :if => :not_redirect?
       end
     end
 
@@ -138,9 +138,8 @@ module Paloma
     end
 
 
-    def html_is_rendered?
-      not_redirect = self.status != 302
-      [nil, 'text/html'].include?(response.content_type) && not_redirect
+    def not_redirect?
+      self.status != 302
     end
 
 
@@ -149,7 +148,10 @@ module Paloma
     #
     def render options = nil, extra_options = {}, &block
       [:json, :js, :xml, :file].each do |format|
-        self.paloma.clear_request if options.has_key?(format)
+        if options.has_key?(format)
+          self.paloma.clear_request
+          break
+        end
       end if options.is_a?(Hash)
 
       super

data/paloma.gemspec

@@ -1,12 +1,13 @@
 Gem::Specification.new do |s|
   s.name        = 'paloma'
-  s.version     = '4.0.0'
+  s.version     = '4.1.0'
   s.summary     = "Provides an easy way to execute page-specific javascript for Rails."
   s.description = "Page-specific javascript for Rails done right"
   s.authors     = ['Karl Paragua']
   s.email       = 'kb.paragua@gmail.com'
   s.files       = `git ls-files`.split("\n")
   s.homepage    = 'https://github.com/kbparagua/paloma'
+  s.license     = 'MIT'
 
   s.add_dependency 'jquery-rails'
 
@@ -16,4 +17,6 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'rspec-rails', ['~> 2.0']
   s.add_development_dependency 'capybara', ['~> 1.0']
   s.add_development_dependency 'jasmine-rails', ['~> 0.4.5']
+  s.add_development_dependency 'turbolinks', ['~> 2.2.2']
+  s.add_development_dependency 'execjs', ['~> 2.1.0']
 end

data/test_app/app/assets/javascripts/application.js

@@ -12,10 +12,15 @@
 //
 //= require jquery
 //= require jquery_ujs
+//= require turbolinks
 //= require paloma
 //= require_tree .
 
 
+// $(document).on('page:restore', function(){
+//   Paloma.executeHook();
+// });
+
 
 //
 //

data/test_app/app/views/layouts/application.html.erb

@@ -1,12 +1,28 @@
 <!DOCTYPE html>
 <html>
 <head>
-  <title>TestApp</title>
+  <title>Paloma Test App</title>
   <%= stylesheet_link_tag    "application", :media => "all" %>
   <%= javascript_include_tag "application" %>
   <%= csrf_meta_tags %>
 </head>
 <body>
+<h1>Paloma Test App</h1>
+
+<div class='links'>
+  <ul>
+    <li><%= link_to 'Main#index', main_index_path %></li>
+    <li><%= link_to 'Main#new', new_main_path %></li>
+    <li><%= link_to 'Main#show', main_path(1) %></li>
+    <li><%= link_to 'Main#edit', edit_main_path(1) %></li>
+    <li><%= link_to 'Main#prevent', prevent_main_index_path %></li>
+    <li><%= link_to 'Main#basic_params', basic_params_main_index_path %></li>
+    <li><%= link_to 'Main#xml_response', xml_response_main_index_path %></li>
+    <li><%= link_to 'Main#file_response', file_response_main_index_path %></li>
+  </ul>
+</div>
+
+<hr/>
 
 <%= yield %>
 

data/test_app/spec/spec_helper.rb

@@ -33,5 +33,5 @@ end
 
 
 def request
-  page.evaluate_script 'Paloma.engine.lastRequest'
+  page.evaluate_script 'Paloma.engine.getRequest()'
 end

data/vendor/assets/javascripts/paloma/engine.js

@@ -2,31 +2,41 @@
 
   var Engine = function(config){
     this.factory = config.factory;
-    this.lastRequest = null;
+    this._request = null;
   };
 
-
-  Engine.prototype.request = function(resource, action, params){
-    this.lastRequest = null;
-
-    var Controller = this.factory.get(resource);
+  Engine.prototype.start = function(){
+    var resource = this._request['controller'],
+        Controller = this.factory.get(resource);
 
     if (!Controller){
       return Paloma.warn('Paloma: undefined controller -> ' + resource);
     }
 
-    var controller = new Controller(params);
+    var controller = new Controller( this._request['params'] ),
+        action = this._request['action'],
+        params = this._request['params'];
 
     if (!controller[action]){
       return Paloma.warn('Paloma: undefined action <' + action +
         '> for <' + resource + '> controller');
     }
 
+
     Paloma.log('Paloma: Execute ' + resource + '#' + action + ' with');
     Paloma.log(params);
 
-    controller[action]();
-    this.lastRequest = {controller: resource, action: action, params: params};
+    controller[ this._request['action'] ]();
+  };
+
+
+  Engine.prototype.setRequest = function(resource, action, params){
+    this._request = {controller: resource, action: action, params: params};
+  };
+
+
+  Engine.prototype.getRequest = function(key){
+    return (!key ? this._request : this._request[key]);
   };
 
 

data/vendor/assets/javascripts/paloma/index.js

@@ -1,6 +1,6 @@
 //= require ./init.js
 //= require ./router.js
 //= require ./controller.js
-//= require ./controller_builder.js
+//= require ./controller_factory.js
 //= require ./engine.js
 //= require ./paloma.js

data/vendor/assets/javascripts/paloma/paloma.js

@@ -16,4 +16,15 @@
 
   Paloma.engine = new Paloma.Engine({factory: Paloma._controllerFactory});
 
+
+  Paloma.executeHook = function(){
+    var $hook = $('.js-paloma-hook:first script:first');
+
+    if ($hook.length == 0){ return; }
+
+    var hook = $hook.html();
+    eval(hook);
+  };
+
+
 })(window.Paloma);

metadata

@@ -1,136 +1,166 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: paloma
-version: !ruby/object:Gem::Version 
-  hash: 63
+version: !ruby/object:Gem::Version
+  version: 4.1.0
   prerelease: 
-  segments: 
-  - 4
-  - 0
-  - 0
-  version: 4.0.0
 platform: ruby
-authors: 
+authors:
 - Karl Paragua
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2014-02-15 00:00:00 +08:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2014-06-07 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: jquery-rails
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: rails
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rails
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 15
-        segments: 
-        - 3
-        - 2
-        - 0
+      - !ruby/object:Gem::Version
         version: 3.2.0
   type: :development
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
-  name: rake
   prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 3.2.0
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id003
-- !ruby/object:Gem::Dependency 
-  name: rspec
   prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id004
-- !ruby/object:Gem::Dependency 
-  name: rspec-rails
   prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec-rails
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 2
-        - 0
-        version: "2.0"
+      - !ruby/object:Gem::Version
+        version: '2.0'
   type: :development
-  version_requirements: *id005
-- !ruby/object:Gem::Dependency 
-  name: capybara
   prerelease: false
-  requirement: &id006 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: capybara
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 15
-        segments: 
-        - 1
-        - 0
-        version: "1.0"
+      - !ruby/object:Gem::Version
+        version: '1.0'
   type: :development
-  version_requirements: *id006
-- !ruby/object:Gem::Dependency 
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
   name: jasmine-rails
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.4.5
+  type: :development
   prerelease: false
-  requirement: &id007 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 5
-        segments: 
-        - 0
-        - 4
-        - 5
+      - !ruby/object:Gem::Version
         version: 0.4.5
+- !ruby/object:Gem::Dependency
+  name: turbolinks
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.2.2
   type: :development
-  version_requirements: *id007
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.2.2
+- !ruby/object:Gem::Dependency
+  name: execjs
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.1.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.1.0
 description: Page-specific javascript for Rails done right
 email: kb.paragua@gmail.com
 executables: []
-
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
+files:
 - .gitignore
 - Changelog.md
 - Gemfile
@@ -196,7 +226,7 @@ files:
 - test_app/script/rails
 - test_app/spec/integration/advanced_spec.rb
 - test_app/spec/integration/basic_spec.rb
-- test_app/spec/javascripts/controller_builder_spec.js
+- test_app/spec/javascripts/controller_factory_spec.js
 - test_app/spec/javascripts/router_spec.js
 - test_app/spec/javascripts/support/jasmine.yml
 - test_app/spec/spec_helper.rb
@@ -212,45 +242,35 @@ files:
 - test_app/vendor/assets/stylesheets/.gitkeep
 - test_app/vendor/plugins/.gitkeep
 - vendor/assets/javascripts/paloma/controller.js
-- vendor/assets/javascripts/paloma/controller_builder.js
+- vendor/assets/javascripts/paloma/controller_factory.js
 - vendor/assets/javascripts/paloma/engine.js
 - vendor/assets/javascripts/paloma/index.js
 - vendor/assets/javascripts/paloma/init.js
 - vendor/assets/javascripts/paloma/paloma.js
 - vendor/assets/javascripts/paloma/router.js
-has_rdoc: true
 homepage: https://github.com/kbparagua/paloma
-licenses: []
-
+licenses:
+- MIT
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.5.2
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Provides an easy way to execute page-specific javascript for Rails.
 test_files: []
-