Discussion :

[BerBiX]

Bonjour à tous,

Je débute sur rails et le développement web de manière générale, et me trouve confronté à un petit problème sans trop savoir quoi faire exactement.

Voilà : j'ai en base une entité Recommendation avec un attribut :content (String) et un attribut :displayed(Boolean).
J'affiche une liste de recommendations en utilisant le partial suivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
<tr>
    <td class="recommendations">
        <span class="content">
            <%= wrap(recommendation.content)%>
        </span>
        <span class="timestamp">
            Posted <%=time_ago_in_words(recommendation.created_at)%> ago.
        </span>
    </td>
    <td>
        <%= form_for Recommendation.find(recommendation) do |f| %>
            <label>Display</label><%= f.check_box :displayed %>
        <% end %>
    </td>
</tr>

Maintenant ce que je voudrais c'est déclencher un update de l'entité en base dès que la check_box est cochée (:displayed => true) ou décochée (:displayed => false). Mais je ne vois pas comment faire.

J'ai vu des personnes conseiller d'utiliser le helper observe_field mais je ne comprends pas comment l'utiliser avec la check_box de la façon dont elle est créée.

D'avance, merci pour votre aide.

[Javix]

D'abord, j'aurais déplacé la ligne Recomandation.find... dans le controller pour garder le modèle MVC sain, - la view n'a rien a voir avec les queries et les choses pareilles. Et dans ta form tu aurais qch de genre:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
<%= form_for @rec do |f|%>
...
<%end%>

Je l'avais fait comme ça dans un des projets:
dans la view:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
<td>
<%= check_box_tag 'rec[displayed]', "1", @rec.displayed, :onclick => toggle_value(@rec) %>
<%= image_tag 'spinner.gif', :id => "spinner-#{@rec.id}",:style => 'display: none' %>
</td>

En suite définit la méthode 'toggle_value' dans un Helper (moi, je l'avais mis dans ApplicationHelper)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
def toggle_value(object)
    remote_function(:url => recomandation_url(:id => object),
      :method   => :put,
      :before => "$('#spinner-#{object.id}').show()",
      :after => "$('#spinner-#{object.id}').hide()",
      :with     => "this.name + '=' + this.checked")
  end

Il ne reste qu'à mettre l'image 'spinner.gif' (voir le fichier ci-attaché) dans ton_app/public/images et c'est tout.

[BerBiX]

Merci pour cette réponse précise.

Concernant :

D'abord, j'aurais déplacé la ligne Recomandation.find... dans le controller pour garder le modèle MVC sain, - la view n'a rien a voir avec les queries et les choses pareilles. Et dans ta form tu aurais qch de genre:
Code :
<%= form_for @rec do |f|%>
...
<%end%>

J'utilise le renderer suivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
<%= render @recommendations %>

pour afficher la liste de recommendations en utilisant le partial "_recommendation.html.erb". Et dans celui-ci la variable à utiliser est "recommendation". Du coup je suis obligé d'utiliser la méthode find(:id) pour retourner une instance à passer à mon form. Je devrais alors créer une méthode dans le application_helper qui me retourne le résultat de Recommendation.find(:id)...

[BerBiX]

Ok, j'y suis presque maintenant avec cependant un dernier problème.

J'ai gardé le check_box_tag comme ceci pour correspondre à mes besoins :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
<td>
    <label>Display</label>
    <%= check_box_tag 'recommendation[displayed]', true, recommendation.displayed, {:onclick =>updated_displayed(recommendation)} %>
</td>

Dans le recommendations_helper j'ai définis la méthode suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
def updated_displayed(recommendation)
    remote_function(  :url => { :action => 'update_displayed', 
                                :controller => 'recommendations', 
                                :id => recommendation.id}, 
                      :with => "'displayed=' + this.checked")
end

Afin d'atteindre la méthode 'update_displayed' du controller il faut ensuite déclarer la route suivante dans le config/routes.rb :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
post 'recommendations/update_displayed/:id', :to => "recommendations#update_displayed"

Et la méthode du controller :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
def update_displayed
    @recommendation.displayed = params[:displayed]
    unless @recommendation.save
      flash[:error] = "Error displaying the recommendation."
    end
    redirect_back_or current_user 
end

Voilà ça fonctionne, mais uniquement lorsque l'on coche la check_box.
Si on la décoche l'update en base n'est pas effectué.

Voici les logs :

Tout d'abord si on coche :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
Processing by RecommendationsController#update_displayed as JS
Parameters: {"authenticity_token"=>"2m7RqiFRVBoNAnOKSzx6m2z6Iof7ZCUkcp+xorhj+HU=", "displayed"=>"true", "id"=>"39"}
Recommendation Load (0.2ms)  SELECT "recommendations".* FROM "recommendations" WHERE ("recommendations"."id" = 39) ORDER BY recommendations.created_at DESC LIMIT 1
User Load (0.2ms)  SELECT "users".* FROM "users" WHERE ("users"."id" = 1) LIMIT 1
CACHE (0.0ms)  SELECT "users".* FROM "users" WHERE ("users"."id" = 1) LIMIT 1
AREL (0.4ms)  UPDATE "recommendations" SET "updated_at" = '2010-12-15 22:18:37.673495', "displayed" = 't' WHERE ("recommendations"."id" = 39)

On voit bien l'appel à la méthode UPDATE.

Maintenant si on décoche :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
Processing by RecommendationsController#update_displayed as JS
Parameters: {"authenticity_token"=>"2m7RqiFRVBoNAnOKSzx6m2z6Iof7ZCUkcp+xorhj+HU=", "displayed"=>"false", "id"=>"39"}
Recommendation Load (0.2ms)  SELECT "recommendations".* FROM "recommendations" WHERE ("recommendations"."id" = 39) ORDER BY recommendations.created_at DESC LIMIT 1
User Load (0.2ms)  SELECT "users".* FROM "users" WHERE ("users"."id" = 1) LIMIT 1
CACHE (0.0ms)  SELECT "users".* FROM "users" WHERE ("users"."id" = 1) LIMIT 1

Plus de méthode UPDATE.

Des idées pourquoi et comment corriger ce comportement ? Encore merci.

[Zfred]

Salut,

Des idées pourquoi et comment corriger ce comportement ?

En fait c'est le comportement normal des check_box, lorsqu'on coche, c'est bon, ça passe, mais quand on décoche la dernière case, Rails ne le voit pas.
Au départ, c'est pas prévu d'utiliser des check_box pour faire du js ou de l'ajax, directement dessus. Il faut utiliser un submit_to_remote, qui va envoyer ta série de coches, et encore je ne suis même pas sûr que ça fonctionne, lorsque toutes les cases sont vides.
Perso, les checkbox j'ai laissé tomber, puisque ça correspond pas au besoin, par contre je sais que certains rajoutent une checkbox cochée, invisible, pour forcer Rails à agir...idée à suivre...si vraiment il faut utiliser check_box.

[BerBiX]

Alors en suivant tous ces conseils j'ai laissé tombé l'utilisation des check_box et je suis passé à un link_to avec un image_tag d'une case cochée/décochée.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
<% if (recommendation.displayed?) %>
    <%= link_to image_tag("checked.png", :border=>0), :action => :hide, :id => recommendation %>
<% else %>
    <%= link_to image_tag("unchecked.png", :border=>0), :action => :display, :id => recommendation %>
<%end%>

Dans le controller :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
  def display
    @recommendation.displayed = true
    unless @recommendation.save
      flash[:error] = "Error displaying the recommendation."
    end
    redirect_back_or current_user 
  end
 
  def hide
    @recommendation.displayed = false
    unless @recommendation.save
      flash[:error] = "Error hiding the recommendation."
    end
    redirect_back_or current_user 
  end

Et j'obtiens un comportement similaire : l'appel à la méthode :display se fait sans problème et génère un UPDATE sur la base. Alors que l'appel à la méthode :hide affiche le flash[:error] et les logs ne présentent aucun appel à la méthode UPDATE.

J'avoue ne pas comprendre...