Download as
Browse Code »

Commit 067a2a2d340a22058f239de75caf2a71ea4e7335

Authored by Lucas Melo
1 parent 1b7dcab6
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

articles, profiles: stop using ajax for some logged-in stuff

Showing 6 changed files with 24 additions and 28 deletions   Show diff stats
app/controllers/public/content_viewer_controller.rb
  Diff comments   View file @067a2a2
@@ -43,11 +43,6 @@ class ContentViewerController < ApplicationController @@ -43,11 +43,6 @@ class ContentViewerController < ApplicationController
43 return 43 return
44 end 44 end
45 45
46 - if request.xhr? && params[:toolbar]  
47 - render :partial => 'article_toolbar'  
48 - return  
49 - end  
50 -  
51 redirect_to_translation if @page.profile.redirect_l10n 46 redirect_to_translation if @page.profile.redirect_l10n
52 47
53 # At this point the page will be showed 48 # At this point the page will be showed
app/views/blocks/profile_image.rhtml
  Diff comments   View file @067a2a2
@@ -14,11 +14,13 @@ @@ -14,11 +14,13 @@
14 <p><%= h block.owner.short_name %></p> 14 <p><%= h block.owner.short_name %></p>
15 <% end %> 15 <% end %>
16 16
17 -<div style="text-align: center; font-size: 75%; clear: both" id="profile-admin-url-<%= block.id %>"></div>  
18 -  
19 -<div class="profile-info-options" id="profile-info-options-<%= block.id %>"></div> 17 +<% if !user.nil? and user.has_permission?('edit_profile', profile) %>
  18 + <div class='admin-link'>
  19 + <%= link_to _('Control panel'), :controller => 'profile_editor' %>
  20 + </div>
  21 +<% end %>
20 22
  23 +<div class="profile-info-options">
  24 + <%= render :file => view_for_profile_actions(block.owner.class) %>
  25 +</div>
21 </div><!-- end class="vcard" --> 26 </div><!-- end class="vcard" -->
22 -<script type="text/javascript">  
23 - <%= remote_function :url => { :controller => 'profile', :profile => profile.identifier, :action => 'profile_info', :block_id => block.id } %>  
24 -</script>  
app/views/blocks/profile_info.rhtml
  Diff comments   View file @067a2a2
@@ -14,13 +14,15 @@ @@ -14,13 +14,15 @@
14 </div> 14 </div>
15 </div> 15 </div>
16 16
17 -<ul class="profile-info-data" id="profile-info-data-<%= block.id %>"> 17 +<ul class="profile-info-data">
18 <li><%= link_to __('Homepage'), block.owner.url, :class => 'url' %></li> 18 <li><%= link_to __('Homepage'), block.owner.url, :class => 'url' %></li>
19 <li><%= link_to _('View profile'), block.owner.public_profile_url %></li> 19 <li><%= link_to _('View profile'), block.owner.public_profile_url %></li>
20 <% if block.owner.enterprise? && !block.owner.environment.enabled?('disable_products_for_enterprises') %> 20 <% if block.owner.enterprise? && !block.owner.environment.enabled?('disable_products_for_enterprises') %>
21 <li><%= link_to(_('Products/Services'), :controller => 'catalog', :profile => block.owner.identifier) %></li> 21 <li><%= link_to(_('Products/Services'), :controller => 'catalog', :profile => block.owner.identifier) %></li>
22 <% end %> 22 <% end %>
23 - <li id="profile-admin-url-<%= block.id %>"></li> 23 + <% if !user.nil? and user.has_permission?('edit_profile', profile) %>
  24 + <li><%= link_to _('Control panel'), :controller => 'profile_editor' %></li>
  25 + <% end %>
24 <% if profile.person? %> 26 <% if profile.person? %>
25 <li><%= _('Since %{year}/%{month}') % { :year => block.owner.created_at.year, :month => block.owner.created_at.month } %></li> 27 <li><%= _('Since %{year}/%{month}') % { :year => block.owner.created_at.year, :month => block.owner.created_at.month } %></li>
26 <% end %> 28 <% end %>
@@ -37,9 +39,8 @@ @@ -37,9 +39,8 @@
37 </div> 39 </div>
38 <% end %> 40 <% end %>
39 41
40 -<div class="profile-info-options" id="profile-info-options-<%= block.id %>"></div> 42 +<div class="profile-info-options">
  43 + <%= render :file => view_for_profile_actions(@block.owner.class) %>
  44 +</div>
41 45
42 </div><!-- end class="vcard" --> 46 </div><!-- end class="vcard" -->
43 -<script type="text/javascript">  
44 - <%= remote_function :url => { :controller => 'profile', :profile => profile.identifier, :action => 'profile_info', :block_id => block.id } %>  
45 -</script>  
app/views/content_viewer/view_page.rhtml
  Diff comments   View file @067a2a2
@@ -14,11 +14,9 @@ @@ -14,11 +14,9 @@
14 window.NO_COMMENT_YET = "<%= _('No comments yet') %>"; 14 window.NO_COMMENT_YET = "<%= _('No comments yet') %>";
15 </script> 15 </script>
16 16
17 -<div id="article-toolbar"></div>  
18 -  
19 -<script type="text/javascript">  
20 - <%= remote_function :update => "article-toolbar", :url => @page.url.merge({ :toolbar => true, :only_path => true }) %>  
21 -</script> 17 +<div id="article-toolbar">
  18 + <%= render :partial => 'article_toolbar' %>
  19 +</div>
22 20
23 <% if @page.display_hits? || @page.license.present? %> 21 <% if @page.display_hits? || @page.license.present? %>
24 <div id='article-sub-header'> 22 <div id='article-sub-header'>
app/views/profile/profile_info.rjs
View file @1b7dcab
@@ -1,6 +0,0 @@ @@ -1,6 +0,0 @@
1 -if !user.nil? and user.has_permission?('edit_profile', profile)  
2 - page.replace_html "profile-admin-url-#{@block.id}", link_to(_('Control panel'), @block.owner.admin_url)  
3 -else  
4 - page.hide "profile-admin-url-#{@block.id}"  
5 -end  
6 -page.replace_html "profile-info-options-#{@block.id}", :file => view_for_profile_actions(@block.owner.class)  
public/designs/themes/base/style.css
  Diff comments   View file @067a2a2
@@ -575,6 +575,12 @@ div#notice { @@ -575,6 +575,12 @@ div#notice {
575 color: #555; 575 color: #555;
576 } 576 }
577 577
  578 +.profile-image-block .admin-link {
  579 + text-align: center;
  580 + font-size: 75%;
  581 + clear: both;
  582 +}
  583 +
578 584
579 /*************************** My Network ******************************/ 585 /*************************** My Network ******************************/
580 586