Rails Ruby on Rails

Comments and changes to this ticket

• You flagged this item as spam.
  

  Rafael Mendonça França September 7th, 2010 @ 06:06 AM

  I could reproduce this issue in rails 3.0.0.

  module AHelper
    def foo
      'A'
    end
  end
  
  module BHelper
    def foo
      'B'
    end
  end
  
  class AController < ...
    def index
      render :inline => '<%= foo %>'
    end
  
  class BController < ...
    def index
      render :inline => '<%= foo %>'
    end
  end
  
  /a => 'B'
  /b => 'B'
  

  If I create the helper using block help in controller the rails render the currect letter, like this

  class AController < ...
    helper { def foo; 'A'; end }
    def index
      render :inline => '<%= foo %>'
    end
  
  class BController < ...
    helper { def foo; 'B'; end }
    def index
      render :inline => '<%= foo %>'
    end
  end
  
  /a => 'A'
  /b => 'B'
  

• You flagged this item as spam.
  

  Rafael Mendonça França September 7th, 2010 @ 06:41 AM

  In rails master this works fine.

  I attached a patch with failing test in 3-0-stable branch.

  • 0001-failing-test-for-helpers-methods-with-same-name.patch 2.3 KB
• 

  David Trasbo September 7th, 2010 @ 12:55 PM

  Rafael,

  This does not work on master either:

  $ script/rails g controller a index
  $ script/rails g controller b index
  

  routes.rb:

  Ticket5565::Application.routes.draw do
    resources :a
    resources :b
  end
  

  Helpers:

  module AHelper
    def foo
      'A'
    end
  end
  
  module BHelper
    def foo
      'B'
    end
  end
  

  In both views:

  <%= foo %>
  

  http://localhost:3000/a => "B" http://localhost:3000/b => "B"

• 

  Rohit Arondekar September 7th, 2010 @ 02:12 PM

  • State changed from “new” to “invalid”
  • Importance changed from “” to “Low”

  You need to do clear_helpers in the conflicting controller like so:

    class AsController < ApplicationController
      clear_helpers
      .
      .
      .
    end
  

  This will remove all existing helpers except the ones defined for that class. In the above case AsHelper

  Related ticket: #5348