Redmine

I will test.

eyal R wrote:

Please consider adding "/" to controller when using url_for or use restful paths.
Thanks.

A patch is welcome, thanks.

Should add: I haven't tested this with Redmine installed on a sub-URI. Will do that... so wait a bit before applying the patch. :)

Hi again,

Using Apache and Passenger, it does work while installed on a sub-URI, as far as I can tell. Also seems to work under WEBrick, if you tweak "config.ru":

require ::File.expand_path('../config/environment',  __FILE__)
map '/redmine' do
  run RedmineApp::Application
end

Cheers...

How about thin and puma?

I don't know... give them a try and let us know!

Another issue linked to namespaced controllers is Menu items that use hashed urls with the controller field like :

  menu.push :my_account, { :controller => 'my', :action => 'account' }, :if => Proc.new { User.current.logged? }

To fix that, you can patch MenuHelper like this :

module MyPlugin
  module MenuHelperPatch
    # Method overriden
    def extract_node_details(node, project=nil)

      item = node
      url = case item.url
      when Hash
        url_hash = item.url.dup

        # Specific : manage controllers in namespaces
        if url_hash.has_key?(:controller) && !url_hash[:controller].starts_with?('/')
          url_hash[:controller] = "/#{url_hash[:controller]}" 
        elsif url_hash.has_key?('controller') && !url_hash['controller'].starts_with?('/')
          url_hash['controller'] = "/#{url_hash['controller']}" 
        end
        # END -- Specific : manage controllers in namespaces

        project.nil? ? url_hash : {item.param => project}.merge(url_hash)
      when Symbol
        if project
          send(item.url, project)
        else
          send(item.url)
        end
      else
        item.url
      end
      caption = item.caption(project)
      return [caption, url, (current_menu_item == item.name)]
    end
  end
end

Redmine::MenuManager::MenuHelper.send(:prepend, MyPlugin::MenuHelperPatch)

• Another issue is the layouts/base template
  It has to be overriden for the search links (/search)