Fix port issue with development server in background

This technique is actually very useful to inject request context into whatever objects you need, and I learnt it from Leandro and Caio.

Fix port issue with development server in background
This technique is actually very useful to inject request context into whatever objects you need, and I learnt it from Leandro and Caio.
Antonio Terceiro
1 parent b5ffea8f
Showing 2 changed files with 6 additions and 9 deletions Show diff stats
lib/noosfero.rb
vendor/plugins/noosfero_urls/init.rb
@@ -68,16 +68,8 @@ module Noosfero
     end
   end
-  # FIXME couldn't think of a way to test this.
-  #
-  # Works (tested by hand) on Rails 2.0.2, with mongrel. Should work with
-  # webrick too.
   def self.development_url_options
-    if Object.const_defined?('OPTIONS')
-      { :port => OPTIONS[:port ]}
-    else
-      {}
-    end
+    @development_url_options || {}
   end
@@ -0,0 +1,5 @@
+if Rails.env == 'development'
+  ActionController::Base.send(:prepend_before_filter) do |controller|
+    Noosfero.instance_variable_set('@development_url_options', { :port => controller.request.port })
+  end
+end
	@@ -68,16 +68,8 @@ module Noosfero		@@ -68,16 +68,8 @@ module Noosfero
68	end	68	end
69	end	69	end
70		70
71	- # FIXME couldn't think of a way to test this.
72	- #
73	- # Works (tested by hand) on Rails 2.0.2, with mongrel. Should work with
74	- # webrick too.
75	def self.development_url_options	71	def self.development_url_options
76	- if Object.const_defined?('OPTIONS')
77	- { :port => OPTIONS[:port ]}
78	- else
79	- {}
80	- end	72	+ @development_url_options \|\| {}
81	end	73	end
82		74
83		75
@@ -0,0 +1,5 @@		@@ -0,0 +1,5 @@
	1	+if Rails.env == 'development'
	2	+ ActionController::Base.send(:prepend_before_filter) do \|controller\|
	3	+ Noosfero.instance_variable_set('@development_url_options', { :port => controller.request.port })
	4	+ end
	5	+end