Merge pull request #360 from linuxfrorg/mastodon_admin_part

Admin part of new Mastodon URL feature

Author: Oumph

app/controllers/admin/accounts_controller.rb

@@ -48,7 +48,9 @@ def update
     else
       @account.inactivate!
       user = @account.user
-      user.homesite = ""
+      user.homesite = nil
+      user.jabber_id = nil
+      user.mastodon_url = nil
       user.save
       Board.amr_notification("Le compte #{@account.login} #{user_url @account.login} a été désactivé par #{current_user.name} #{user_url(current_user)}")
       redirect_back notice: "Compte désactivé", fallback_location: admin_accounts_url
@@ -58,7 +60,9 @@ def update
   def destroy
     @account.inactivate!
     user = @account.user
-    user.homesite = ""
+    user.homesite = nil
+    user.jabber_id = nil
+    user.mastodon_url = nil
     user.save
     Board.amr_notification("Le compte #{@account.login} #{user_url @account.login} a été désactivé par #{current_user.name} #{user_url(current_user)}")
     redirect_to admin_accounts_url, notice: "Compte désactivé"

app/helpers/users_helper.rb

@@ -42,6 +42,8 @@ def jabber_link(user)
 
   def mastodon_link(user)
     return if user.mastodon_url.blank?
+    karma = user.account.try(:karma).to_i
+    return unless karma > 0
     link_to("Mastodon", user.mastodon_url)
   end
 end

app/views/admin/accounts/index.html.haml

@@ -11,7 +11,7 @@
 
 %table
   %tr
-    %th Pseudo / Courriel
+    %th Pseudo / Adresses
     %th Karma
     %th Adresse IP courante / précédente
     %th Commentaires / contenus / étiquettes
@@ -28,9 +28,15 @@
           = image_tag "/images/icones/check.svg", alt: "Actif", title: "Actif", width: "16px"
         %br
         = account.email
-        - if account.user.homesite
+        - if account.user.homesite.present?
           %br
           = account.user.homesite
+        - if account.user.jabber_id.present?
+          %br
+          = account.user.jabber_id
+        - if account.user.mastodon_url.present?
+          %br
+          = account.user.mastodon_url
       %td= account.karma
       %td
         - if account.current_sign_in_ip != account.last_sign_in_ip

app/views/devise/registrations/_preferences.html.haml

@@ -13,7 +13,7 @@
       = u.text_field :jabber_id, placeholder: "me@example.net", maxlength: 32
     %p
       = u.label :mastodon_url, 'Adresse du compte Mastodon'
-      = u.url_field :mastodon_url, placeholder: "https://mastodon.example/@me", maxlength: 255
+      = u.url_field :mastodon_url, placeholder: "https://mastodon.example/@me", maxlength: 100
       %span.help
         Après avoir enregistré l’adresse de votre compte Mastodon, vous pourrez utiliser le lien <kbd>#{user_url(current_user)}</kbd> pour
         <a href="https://docs.joinmastodon.org/user/profile/#verification">prouver que le compte Mastodon et le compte LinuxFr.org appartiennent à la même personne.</a>

app/views/users/_recent.html.haml

@@ -11,6 +11,12 @@
       %li Dernière connexion : #{a.current_sign_in_at ? l(@user.account.current_sign_in_at) : "-"}
       %li Dernière action : #{a.updated_at ? l(@user.account.updated_at) : "-"}
       %li Karma : #{a.karma} (minimum : #{a.min_karma}, maximum : #{a.max_karma})
+      - if @user.homesite.present?
+        %li Site perso : #{@user.homesite}
+      - if @user.jabber_id.present?
+        %li XMPP : #{@user.jabber_id}
+      - if @user.mastodon_url.present?
+        %li Mastodon : #{@user.mastodon_url}
     - if current_account.admin?
       %p Visibilité
       %ul

app/views/users/_user.html.haml

@@ -6,9 +6,12 @@
   - unless user.account && user.account.active_for_authentication?
     %p
       Compte fermé
-  - if user.homesite
+  - if user.homesite.present?
     %p
       Site Web personnel : #{link_to user.homesite, user.homesite, class: 'url me'}
-  - if current_account && user.jabber_id
+  - if current_account && user.jabber_id.present?
     %p
       Adresse XMPP : #{link_to 'xmpp:' + user.jabber_id, 'xmpp:' + user.jabber_id, class: 'url'}
+  - if current_account && user.mastodon_url.present?
+    %p
+      Adresse Mastodon : #{link_to 'Mastodon', user.mastodon_url, class: 'url'}

app/views/users/show.html.haml

@@ -4,7 +4,7 @@
 
 =h1 "#{@user.name} a écrit #{pluralize @nodes.total_count, "contenu"} de type dépêche ou journal"
 - feed "Flux Atom de #{@user.name}", "/users/#{@user.to_param}.atom"
-- if @user.mastodon_url
+- if @user.mastodon_url.present? and @user.account.try(:karma).to_i > 0
   - link "me", @user.mastodon_url
 - filter_pagination_params ['id', 'order', 'page']
 = paginated_nodes @nodes

db/pages/aide.html

@@ -297,7 +297,7 @@ <h2 id="aide-karma">Système de karma <a href="#aide-karma" class="anchor">¶</a
 
 <p>Pour pouvoir écrire un journal, publier un lien ou discuter sur la tribune, il faut avoir un karma supérieur strictement à 0, et pour éditer une page de wiki, il faut un karma strictement supérieur à 20 (sources <a href="https://github.com/linuxfrorg/linuxfr.org/blob/master/app/models/diary.rb">journal</a>, <a href="https://github.com/linuxfrorg/linuxfr.org/blob/master/app/models/bookmark.rb">lien</a>, <a href="https://github.com/linuxfrorg/linuxfr.org/blob/master/app/models/wiki_page.rb">wiki</a> et <a href="https://github.com/linuxfrorg/linuxfr.org/blob/master/app/models/account.rb">tribune</a>).</p>
 
-<p>Pour pouvoir afficher l’adresse de son site personnel, il faut avoir un karma strictement supérieur à 0 (<a href="https://github.com/linuxfrorg/linuxfr.org/blob/master/app/helpers/users_helper.rb">source</a>).</p>
+<p>Pour pouvoir afficher l’adresse de son site personnel ou de son compte Mastodon, il faut avoir un karma strictement supérieur à 0 (<a href="https://github.com/linuxfrorg/linuxfr.org/blob/master/app/helpers/users_helper.rb">source</a>).</p>