Merge branch 'master' into next

Antonio Terceiro
2 parents d818aa8e cb62ddad
Showing 9 changed files with 70 additions and 93 deletions Show diff stats
app/models/block.rb
app/models/highlights_block.rb
app/views/user_mailer/profiles_suggestions_email.html.erb
features/signup.feature
plugins/event/po/pt/event.po
script/production
test/unit/block_test.rb
test/unit/highlights_block_test.rb
test/unit/user_mailer_test.rb
@@ -23,7 +23,7 @@ class Block &lt; ActiveRecord::Base
   end
   def get_limit
-    [0,limit].max
+    [0,limit.to_i].max
   end
   def embed_code
@@ -26,8 +26,16 @@ class HighlightsBlock &lt; Block
   end
   def featured_images
-    block_images = images.select{|i| !i[:image_src].nil? }.sort { |x, y| x[:position] <=> y[:position] }
-    shuffle ? block_images.shuffle : block_images
+    images = get_images
+    shuffle ? images.shuffle : images
+  end
+
+  def get_images
+    images.select do |i|
+      !i[:image_src].nil?
+    end.sort do |x, y|
+      x[:position] <=> y[:position]
+    end
   end
   def content(args={})
@@ -10,8 +10,8 @@
     <% @people_suggestions.each do |person| %>
     <li><a href="<%= url_for(person.public_profile_url) %>"><%= person.name %></a></li>
     <% end %>
-  <ul>
-  <%= _("To see the full list of friends suggestions, follow the link: %s") % @people_suggestions_url %>
+  </ul>
+  <%= _("To see the full list of friends suggestions, follow the link: %s") % url_for(@people_suggestions_url) %>
 <% end %>
 <% unless @communities_suggestions.empty? %>
@@ -21,13 +21,15 @@
     <% @communities_suggestions.each do |community| %>
     <li><a href="<%= url_for(community.public_profile_url) %>"><%= community.name %></a></li>
     <% end %>
-  <ul>
-  <%= _("To see the full list of communities suggestions, follow the link: %s") % @communities_suggestions_url %>
+  </ul>
+  <%= _("To see the full list of communities suggestions, follow the link: %s") % url_for(@communities_suggestions_url) %>
 <% end %>
 <%= _("Greetings,") %>
---
-<%= _('%s team.') % @environment %>
+<p><%= _("Greetings,") %></p>
+
+--<br/>
+<%= _('%s team.') % @environment %><br/>
 <%= @url %>
@@ -267,32 +267,6 @@ Feature: signup
     Then I should be on the homepage
   @selenium
-  Scenario: join community on signup
-    Given the following users
-      | login | name |
-      | mariasilva | Maria Silva |
-    And the following communities
-       | name           | identifier    | owner     |
-       | Free Software  | freesoftware  | mariasilva |
-    And feature "skip_new_user_email_confirmation" is disabled on environment
-    And I am on /freesoftware
-    When I follow "Join"
-    And I follow "New user"
-    And I fill in the following within ".no-boxes":
-      | e-Mail                | josesilva@example.com |
-      | Username              | josesilva             |
-      | Password              | secret                |
-      | Password confirmation | secret                |
-      | Full name             | José da Silva         |
-    And wait for the captcha signup time
-    And I press "Create my account"
-    And I go to josesilva's confirmation URL
-    And I fill in "Username" with "josesilva"
-    And I fill in "Password" with "secret"
-    And I press "Log in"
-    Then "José da Silva" should be a member of "Free Software"
-
-  @selenium
   Scenario: user registration is moderated by admin
     Given feature "admin_must_approve_new_users" is enabled on environment
     And feature "skip_new_user_email_confirmation" is disabled on environment
@@ -5,8 +5,8 @@
 #
 msgid ""
 msgstr ""
-"Project-Id-Version: 1.0-690-gcb6e853\n"
-"POT-Creation-Date: 2015-03-05 12:09-0300\n"
+"Project-Id-Version: 1.1~rc4\n"
+"POT-Creation-Date: 2015-04-20 19:44-0300\n"
 "PO-Revision-Date: 2015-01-30 00:18-0000\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language-Team: LANGUAGE <LL@li.org>\n"
@@ -16,61 +16,53 @@ msgstr &quot;&quot;
 "Content-Transfer-Encoding: 8bit\n"
 "Plural-Forms: nplurals=INTEGER; plural=EXPRESSION;\n"
+#: plugins/event/lib/event_plugin.rb:4
+msgid "Event Extras"
+msgstr "Extras para Eventos"
+
+#: plugins/event/lib/event_plugin.rb:8
+msgid ""
+"Include a new block to show the environment's or profiles' events information"
+msgstr ""
+"Adiciona um novo bloco para apresentar as informações de eventos do ambiente "
+"ou dos perfis"
+
 #: plugins/event/lib/event_plugin/event_block.rb:12
 msgid "Events"
 msgstr "Eventos"
 #: plugins/event/lib/event_plugin/event_block.rb:16
 msgid "Show the profile events or all environment events."
-msgstr "Mostrar todos os eventos."
+msgstr "Mostra todos os eventos de um perfil ou do ambiente."
 #: plugins/event/lib/event_plugin/event_block.rb:65
-#, fuzzy
 msgid "One month ago"
 msgid_plural "%d months ago"
-msgstr[0] "Iniciou a um mês atrás."
-msgstr[1] "Iniciou a %d meses atrás."
+msgstr[0] "Um mês atrás"
+msgstr[1] "%d meses atrás."
 #: plugins/event/lib/event_plugin/event_block.rb:67
 msgid "Yesterday"
 msgid_plural "%d days ago"
-msgstr[0] ""
-msgstr[1] ""
+msgstr[0] "Ontem"
+msgstr[1] "%d dias atrás"
 #: plugins/event/lib/event_plugin/event_block.rb:69
 msgid "Today"
-msgstr ""
+msgstr "Hoje"
 #: plugins/event/lib/event_plugin/event_block.rb:71
 msgid "Tomorrow"
 msgid_plural "%d days left to start"
-msgstr[0] ""
-msgstr[1] ""
+msgstr[0] "Amanhã"
+msgstr[1] "%d dias para começar"
 #: plugins/event/lib/event_plugin/event_block.rb:73
-#, fuzzy
 msgid "One month left to start"
 msgid_plural "%d months left to start"
 msgstr[0] "Um mês para iniciar"
 msgstr[1] "% meses para iniciar"
-#: plugins/event/lib/event_plugin.rb:4
-msgid "Event Extras"
-msgstr "Eventos"
-
-#: plugins/event/lib/event_plugin.rb:8
-msgid ""
-"Include a new block to show the environment's or profiles' events information"
-msgstr ""
-"Adiciona um novo bloco para apresentar as informações de eventos do ambiente "
-"ou de perfis"
-
-#: plugins/event/views/event_plugin/event_block_item.html.erb:6
-msgid "Duration: 1 day"
-msgid_plural "Duration: %s days"
-msgstr[0] ""
-msgstr[1] ""
-
 #: plugins/event/views/profile_design/event_plugin/_event_block.html.erb:1
 msgid "Limit of items"
 msgstr "Limite de itens"
@@ -94,12 +86,17 @@ msgstr &quot;Mostrar apenas eventos futuros&quot;
 #: plugins/event/views/profile_design/event_plugin/_event_block.html.erb:22
 msgid "Limit of days to display"
-msgstr "Limite de dias de distância para mostrar eventos"
+msgstr "Limite de dias para mostrar"
 #: plugins/event/views/profile_design/event_plugin/_event_block.html.erb:24
-#, fuzzy
 msgid "Only show events in this interval of days."
-msgstr "Mostar somente eventos que acontecem dentro do limite de dias"
+msgstr "Mostar somente os eventos nesse intervalo de dias"
+
+#: plugins/event/views/event_plugin/event_block_item.html.erb:6
+msgid "Duration: 1 day"
+msgid_plural "Duration: %s days"
+msgstr[0] "Duração: 1 dia"
+msgstr[1] "Duração: %s dias"
 #~ msgid "Started one day ago."
 #~ msgid_plural "Started %d days ago."
@@ -85,7 +85,7 @@ environments_loop() {
 }
 do_running() {
-  pids=$(sed "s/.*/& /" tmp/pids/thin.*.pid | tr -d '\n' 2>/dev/null || true)
+  pids=$(sed "s/.*/& /" tmp/pids/thin.*.pid 2>/dev/null | tr -d '\n')
   # passes if any of $pids exist, fails otherwise
   kill -0 $pids > /dev/null 2>&1
 }
@@ -330,4 +330,11 @@ class BlockTest &lt; ActiveSupport::TestCase
     block.save!
     assert !block.display_to_user?(person_friend)
   end
+
+  should 'get limit as a number when limit is string' do
+    block = RecentDocumentsBlock.new
+    block.settings[:limit] = '5'
+    assert block.get_limit.is_a?(Fixnum)
+  end
+
 end
@@ -109,33 +109,14 @@ class HighlightsBlockTest &lt; ActiveSupport::TestCase
   end
   should 'list images randomically' do
-    f1 = mock()
-    f1.expects(:public_filename).returns('address')
-    UploadedFile.expects(:find).with(1).returns(f1)
-    f2 = mock()
-    f2.expects(:public_filename).returns('address')
-    UploadedFile.expects(:find).with(2).returns(f2)
-    f3 = mock()
-    f3.expects(:public_filename).returns('address')
-    UploadedFile.expects(:find).with(3).returns(f3)
-    f4 = mock()
-    f4.expects(:public_filename).returns('address')
-    UploadedFile.expects(:find).with(4).returns(f4)
-    f5 = mock()
-    f5.expects(:public_filename).returns('address')
-    UploadedFile.expects(:find).with(5).returns(f5)
     block = HighlightsBlock.new
-    i1 = {:image_id => 1, :address => '/address', :position => 3, :title => 'address'}
-    i2 = {:image_id => 2, :address => '/address', :position => 1, :title => 'address'}
-    i3 = {:image_id => 3, :address => '/address', :position => 2, :title => 'address'}
-    i4 = {:image_id => 4, :address => '/address', :position => 5, :title => 'address'}
-    i5 = {:image_id => 5, :address => '/address', :position => 4, :title => 'address'}
-    block.images = [i1,i2,i3,i4,i5]
     block.shuffle = true
-    block.save!
-    block.reload
-    assert_equal [i1,i2,i3,i4,i5], block.images
-    assert_not_equal [i2,i3,i1,i4,i5], block.featured_images
+
+    images = []
+    block.expects(:get_images).returns(images)
+    images.expects(:shuffle).returns(images)
+
+    block.featured_images
   end
   [Environment, Profile].each do |klass|
@@ -18,6 +18,14 @@ class UserMailerTest &lt; ActiveSupport::TestCase
     end
   end
+  should 'deliver profiles suggestions email' do
+    person = create_user('some-user').person
+    ProfileSuggestion.create!(:person => person, :suggestion =>
+fast_create(Person))
+    email = UserMailer.profiles_suggestions_email(person).deliver
+    assert_match /profile\/some-user\/friends\/suggest/, email.body.to_s
+  end
+
   private
     def read_fixture(action)
	@@ -23,7 +23,7 @@ class Block < ActiveRecord::Base		@@ -23,7 +23,7 @@ class Block < ActiveRecord::Base
23	end	23	end
24		24
25	def get_limit	25	def get_limit
26	- [0,limit].max	26	+ [0,limit.to_i].max
27	end	27	end
28		28
29	def embed_code	29	def embed_code
	@@ -26,8 +26,16 @@ class HighlightsBlock < Block		@@ -26,8 +26,16 @@ class HighlightsBlock < Block
26	end	26	end
27		27
28	def featured_images	28	def featured_images
29	- block_images = images.select{\|i\| !i[:image_src].nil? }.sort { \|x, y\| x[:position] <=> y[:position] }
30	- shuffle ? block_images.shuffle : block_images	29	+ images = get_images
		30	+ shuffle ? images.shuffle : images
		31	+ end
		32	+
		33	+ def get_images
		34	+ images.select do \|i\|
		35	+ !i[:image_src].nil?
		36	+ end.sort do \|x, y\|
		37	+ x[:position] <=> y[:position]
		38	+ end
31	end	39	end
32		40
33	def content(args={})	41	def content(args={})
	@@ -267,32 +267,6 @@ Feature: signup		@@ -267,32 +267,6 @@ Feature: signup
267	Then I should be on the homepage	267	Then I should be on the homepage
268		268
269	@selenium	269	@selenium
270	- Scenario: join community on signup
271	- Given the following users
272	- \| login \| name \|
273	- \| mariasilva \| Maria Silva \|
274	- And the following communities
275	- \| name \| identifier \| owner \|
276	- \| Free Software \| freesoftware \| mariasilva \|
277	- And feature "skip_new_user_email_confirmation" is disabled on environment
278	- And I am on /freesoftware
279	- When I follow "Join"
280	- And I follow "New user"
281	- And I fill in the following within ".no-boxes":
282	- \| e-Mail \| josesilva@example.com \|
283	- \| Username \| josesilva \|
284	- \| Password \| secret \|
285	- \| Password confirmation \| secret \|
286	- \| Full name \| José da Silva \|
287	- And wait for the captcha signup time
288	- And I press "Create my account"
289	- And I go to josesilva's confirmation URL
290	- And I fill in "Username" with "josesilva"
291	- And I fill in "Password" with "secret"
292	- And I press "Log in"
293	- Then "José da Silva" should be a member of "Free Software"
294	-
295	- @selenium
296	Scenario: user registration is moderated by admin	270	Scenario: user registration is moderated by admin
297	Given feature "admin_must_approve_new_users" is enabled on environment	271	Given feature "admin_must_approve_new_users" is enabled on environment
298	And feature "skip_new_user_email_confirmation" is disabled on environment	272	And feature "skip_new_user_email_confirmation" is disabled on environment
	@@ -85,7 +85,7 @@ environments_loop() {		@@ -85,7 +85,7 @@ environments_loop() {
85	}	85	}
86		86
87	do_running() {	87	do_running() {
88	- pids=$(sed "s/./& /" tmp/pids/thin..pid \| tr -d '\n' 2>/dev/null \|\| true)	88	+ pids=$(sed "s/./& /" tmp/pids/thin..pid 2>/dev/null \| tr -d '\n')
89	# passes if any of $pids exist, fails otherwise	89	# passes if any of $pids exist, fails otherwise
90	kill -0 $pids > /dev/null 2>&1	90	kill -0 $pids > /dev/null 2>&1
91	}	91	}
	@@ -330,4 +330,11 @@ class BlockTest < ActiveSupport::TestCase		@@ -330,4 +330,11 @@ class BlockTest < ActiveSupport::TestCase
330	block.save!	330	block.save!
331	assert !block.display_to_user?(person_friend)	331	assert !block.display_to_user?(person_friend)
332	end	332	end
		333	+
		334	+ should 'get limit as a number when limit is string' do
		335	+ block = RecentDocumentsBlock.new
		336	+ block.settings[:limit] = '5'
		337	+ assert block.get_limit.is_a?(Fixnum)
		338	+ end
		339	+
333	end	340	end
	@@ -109,33 +109,14 @@ class HighlightsBlockTest < ActiveSupport::TestCase		@@ -109,33 +109,14 @@ class HighlightsBlockTest < ActiveSupport::TestCase
109	end	109	end
110		110
111	should 'list images randomically' do	111	should 'list images randomically' do
112	- f1 = mock()
113	- f1.expects(:public_filename).returns('address')
114	- UploadedFile.expects(:find).with(1).returns(f1)
115	- f2 = mock()
116	- f2.expects(:public_filename).returns('address')
117	- UploadedFile.expects(:find).with(2).returns(f2)
118	- f3 = mock()
119	- f3.expects(:public_filename).returns('address')
120	- UploadedFile.expects(:find).with(3).returns(f3)
121	- f4 = mock()
122	- f4.expects(:public_filename).returns('address')
123	- UploadedFile.expects(:find).with(4).returns(f4)
124	- f5 = mock()
125	- f5.expects(:public_filename).returns('address')
126	- UploadedFile.expects(:find).with(5).returns(f5)
127	block = HighlightsBlock.new	112	block = HighlightsBlock.new
128	- i1 = {:image_id => 1, :address => '/address', :position => 3, :title => 'address'}
129	- i2 = {:image_id => 2, :address => '/address', :position => 1, :title => 'address'}
130	- i3 = {:image_id => 3, :address => '/address', :position => 2, :title => 'address'}
131	- i4 = {:image_id => 4, :address => '/address', :position => 5, :title => 'address'}
132	- i5 = {:image_id => 5, :address => '/address', :position => 4, :title => 'address'}
133	- block.images = [i1,i2,i3,i4,i5]
134	block.shuffle = true	113	block.shuffle = true
135	- block.save!
136	- block.reload
137	- assert_equal [i1,i2,i3,i4,i5], block.images
138	- assert_not_equal [i2,i3,i1,i4,i5], block.featured_images	114	+
		115	+ images = []
		116	+ block.expects(:get_images).returns(images)
		117	+ images.expects(:shuffle).returns(images)
		118	+
		119	+ block.featured_images
139	end	120	end
140		121
141	[Environment, Profile].each do \|klass\|	122	[Environment, Profile].each do \|klass\|
	@@ -18,6 +18,14 @@ class UserMailerTest < ActiveSupport::TestCase		@@ -18,6 +18,14 @@ class UserMailerTest < ActiveSupport::TestCase
18	end	18	end
19	end	19	end
20		20
		21	+ should 'deliver profiles suggestions email' do
		22	+ person = create_user('some-user').person
		23	+ ProfileSuggestion.create!(:person => person, :suggestion =>
		24	+fast_create(Person))
		25	+ email = UserMailer.profiles_suggestions_email(person).deliver
		26	+ assert_match /profile\/some-user\/friends\/suggest/, email.body.to_s
		27	+ end
		28	+
21	private	29	private
22		30
23	def read_fixture(action)	31	def read_fixture(action)