Bonjour à tous.

Voilà le contexte. J'utilise Redmine 3.3.3, basé sur Ruby On Rails.
Afin de réaliser des cartes et de gérer des issues géoréférencée, un table "pointes" permet pour chaque Issue d'être géolocalisée.
Cette table contient l'id de l'Issue, l'id du pointé, un point (base de données spatiale postgresql), un état et diverses informations.
Afin de visualiser ces coordonnées mais aussi de pouvoir en modifier les informations, plusieurs plugins ont été développés (permettant d'ajouter / supprimer un plusieurs pointés par FT via des requêtes rest).
Pour afficher les informations des pointés, un fichier a été ajouté dans <nom du plugin>/lib/<nom du plugin>/view_hook_listeners.rb
Ceci va exécuter le fichier app/views/<nom du plugin>/issue/_issus_details.html.erb pour injecter du code dans la page de visualisation de l'Issue et ça marche très bien.

view_hook_listeners.rb :
module GeometryStatus
  # Voir : http://www.redmine.org/projects/redmine/wiki/Hooks
  class Hooks < Redmine::Hook::ViewListener
    # rewrite select for trackers on issue form // view_issues_show_details_bottom
    def view_issues_show_details_bottom(context={})
      issue = context[:issue]
      project = context[:project]

      --- du code non copié ici car un peu complexe ---
        # charger le fichier app\views\geometry_status\issues\_issue_details.html.erb
        return context[:controller].send(:render_to_string, partial: 'geometry_status/issues/issue_details', locals: { project: project, issue: issue, pointes: pointes, portee_du_defaut: portee_du_defaut, port: qgis_plugin_port?, chantier_name: chantier_name, geomaps_map_server_login: geomaps_map_server_login?, geomaps_map_server_password: geomaps_map_server_password?, geomaps_map_server_url: geomaps_map_server_url?, nb_status_total: nb_status_total, nb_status_to_validated: nb_status_to_validated, nb_status_resolved: nb_status_resolved, nb_status_refused: nb_status_refused })
      end
      ''
    end
  end
end

Ce que je veux

Le formulaire affichant les informations des pointés a été injecté en javascript dans la page de détails de l'Issue et fonctionne (la sauvegarde des modifications est faite en AJAX). Toutefois, afin d'obtenir une meilleur intégration dans Redmine j'ai souhaité non pas utiliser un formulaire normal avec form mais le faire en rails (sinon ça pose des problèmes de détection de champs modifiés non sauvé (ou sauvé) mal pris en charge par Rails puisqu'on ne passe pas par le framework) :
Le code suivant est placé dans une boucle, en effet il peut y avoir plusieurs instances de 'pointé' par FT, chaque instance ayant son formulaire propre.

        <%= form_for :pointe, html: {class: 'geometry_pointe_form'}  do |form| %>
          <%= form.hidden_field :id, value: pointe.id %>
                <legend><strong><%= l(:geometry_legend_status) %></strong></legend>
                <%= form.select(:statut, options_for_select([[l(:geometry_status_new), 1],
                                                            [l(:geometry_status_refused), 2],
                                                            [l(:geometry_status_accepted), 3],
                                                            [l(:geometry_status_to_validated), 4],
                                                            [l(:geometry_status_resolved), 5],
                                                            [l(:geometry_status_confirmed), 6]
                                                           ],
                                                           pointe.statut)) %>
               .... les autres  champs .....

          <%= form.submit "Submit" %>
        <% end %>

Ce que j'obtiens

J'obient exactement ce que je veux au niveau affichage, bien entendu le code est bien plus complexe.
Par contre impossible de savoir quoi mettre dans quel fichier pour capter le submit et sauver en base de données les informations modifiées.
J'obtiens "Page not found" systématiquement lorsque je clique sur le bouton submit.

J'ai bien tenté dans app/status/controller/pointe_controller.rb de mettre le code suivant mais sans aucun effet.
Et quand ça ne fonctionne pas, on peut pas dire de rails nous aide à comprendre les choses.

class PointeController < ApplicationController
  def create
    Pointes.create(params[:id])
  end

  def update
    @pointe = Pointes.find(params[:id]]))
    if @pointe.update(pointe_params)
      redirect_to 'pages/success'
    else
      redirect_to 'pages/error'
    end
  end

private
  def pointe_params
   params.require(:id).permit(:comment, :statut)
  end  
end

Voilà, j'ai passé deux jours à porter de façon propre le formulaire que j'avais fait en javascript exclusivement, mais maintenant il faut que je le sauve ce(s) sacré(s) formulaire(s) !

Merci pour vos réponses.

3 réponses


Hello,

Est-ce que tu sais si l'erreur que tu as vient de l'url pour le formulaire ?
Dans ce cas là, il faut peut-être voir à utiliser le champs url de form_for.

Sinon, peut-être que ton update est bien appelé, mais qu'il ne trouve pas les redirect que tu lui donnes (pages/success ou pages/error)

Feneck91
Auteur

Salut et merci pour ta réponse.
Actuellement, ça fonctionne mais c'est bof bof...
1> Dans l'update il n'y a pas de redirect et déjà c'est peut-être ça qui manque, mais je ne sais pas quoi mettre comme redirect comme page (attention je suis dans un plugin sous Redmine). En fait pour le moment, toutes les modification en base sont envoyée via rest dans des plugins écris en python (plugin QGis).
2> Comme il n'y a pas de redirect, on ne voit pas si la sauvegarde a fonctionnée ou pas, en fait ça n'a choqué personne puisqu'en rest je n'utilise que le code retour.
3> Même si la sauvegarde échoue, le formulaire est considéré comme sauvé par le framework car si on change de page, il n'y a plus d'avertissement comme quoi des informations ont changées, bizarre...

Actuellement j'appel un javascript qui utilise l'API rest et si ça fonctionne alors je laisse le traitement par défaut (qui va sauvegarder une seconde fois le formulaire), sinon je bloque le traitement par défaut du bouton et donc rails le considère comme non sauvé.

Mon controller (simplifié) :

class PointesController < ApplicationController
  unloadable

  include ActionController::Live
  include ActionController::RespondWith
  include ActionController::MimeResponds

  before_filter :find_current_user
  respond_to :xml, :json

  #
  # show pointe by id
  #
  def show
    @pointes = Pointes.find(params[:id])
    respond_with @pointes
  end

  #
  # create pointe
  #
  def create
    @pointe = Pointes.new(params[:pointes])
    if @pointe.save
      respond_with(@pointe, status: 201, location: nil)
    else
      respond_with(@pointe.errors, location: nil)
    end
  end

  #
  # update pointe
  #
  def update
    @pointe = Pointes.find(params[:id])
    if @pointe.update(params[:pointes])
      respond_with({:msg => "Pointé mis à jour"}, status: 201, location: nil)
    else
      respond_with({:msg => @pointe.errors}, location: nil)
    end
  end

  #
  # delete pointe
  #
  def delete
    Pointes.find(params[:id]).destroy
    respond_with({:msg => "Pointé supprimé"}, status: 201, location: nil)
  end

Le code du formulaire :

        <%= form_for :pointes, url: '../geometry/%d' % pointe.id, format: :json, remote: true, html: {class: 'geometry_pointe_form'}  do |form| %>
          <%= form.hidden_field :id, value: pointe.id %>
          <fieldset>
            <!-- Title -->
            <legend><strong><%= l(:pointe_num_id, :index => index + 1, :id=> pointe.id) %></strong></legend>
            <div class="geometry_pointe_column1">
              <br/>
              <% if pointe.pointe != nil && pointe.pointe.as_text().length != 0 %>
                <!-- WKT of the pointe -->
                <div class="bubble" id="pointe_geometry_copy_wkt_ok_<%= pointe.id %>" style="display:none;">
                  <div class="bubble-text">
                    <p><%= l(:text_copied_ok) %></p>
                   </div>
                </div>
                <div class="bubblenok" id="pointe_geometry_copy_wkt_nok_<%= pointe.id %>" style="display:none">
                  <div class="bubblenok-text">
                    <p><%= l(:text_copied_nok) %></p>
                   </div>
                </div>
                <button class="geometry_light_button" onClick="copy_clipboard_WKT(event, '<%= pointe.pointe.as_text() %>', 'pointe_geometry_copy_wkt_ok_<%= pointe.id %>', 'pointe_geometry_copy_wkt_nok_<%= pointe.id %>')" title="<%= l(:geometry_clic_button_copy_wkt_to_clipboard) %>">
                  <%= image_tag("copy_clipboard_wkt.png", :plugin => "geometry_status", :title => l(:geometry_clic_button_copy_wkt_to_clipboard), :align => "center") %>
                </button>
                <!-- Text of the pointe -->
                <div class="bubble" id="pointe_geometry_copy_ok_<%= pointe.id %>" style="display:none;">
                  <div class="bubble-text">
                    <p><%= l(:text_copied_ok) %></p>
                   </div>
                </div>
                <div class="bubblenok" id="pointe_geometry_copy_nok_<%= pointe.id %>" style="display:none">
                  <div class="bubblenok-text">
                    <p><%= l(:text_copied_nok) %></p>
                   </div>
                </div>
                <button class="geometry_light_button" onClick="copy_clipboard_WKT(event, '<%= pointe.pointe.x() %> <%= pointe.pointe.y() %>', 'pointe_geometry_copy_ok_<%= pointe.id %>', 'pointe_geometry_copy_nok_<%= pointe.id %>')" title="<%= l(:geometry_clic_button_copy_to_clipboard) %>">
                  <%= image_tag("copy_clipboard.png", :plugin => "geometry_status", :title => l(:geometry_clic_button_copy_to_clipboard), :align => "center") %>
                </button>
                <%= l(:geometry_pointe) %>
                <br/>
              <% end %>
              <% if pointe.point_attendu != nil && pointe.point_attendu.as_text().length != 0 %>
                <!-- WKT of the expected point -->
                <div class="bubble" id="expected_point_geometry_copy_wkt_ok_<%= pointe.id %>" style="display:none;">
                  <div class="bubble-text">
                    <p><%= l(:text_copied_ok) %></p>
                   </div>
                </div>
                <div class="bubblenok" id="expected_point_geometry_copy_wkt_nok_<%= pointe.id %>" style="display:none">
                  <div class="bubblenok-text">
                    <p><%= l(:text_copied_nok) %></p>
                   </div>
                </div>
                <button class="geometry_light_button" onClick="copy_clipboard_WKT(event, '<%= pointe.point_attendu.as_text() %>', 'expected_point_geometry_copy_wkt_ok_<%= pointe.id %>', 'expected_point_geometry_copy_wkt_nok_<%= pointe.id %>')" title="<%= l(:geometry_clic_button_copy_wkt_to_clipboard) %>">
                  <%= image_tag("copy_clipboard_wkt.png", :plugin => "geometry_status", :title => l(:geometry_clic_button_copy_wkt_to_clipboard), :align => "center") %>
                </button>
                <!-- Text of the expected point -->
                <div class="bubble" id="expected_point_geometry_copy_ok_<%= pointe.id %>" style="display:none;">
                  <div class="bubble-text">
                    <p><%= l(:text_copied_ok) %></p>
                   </div>
                </div>
                <div class="bubblenok" id="expected_point_geometry_copy_nok_<%= pointe.id %>" style="display:none">
                  <div class="bubblenok-text">
                    <p><%= l(:text_copied_nok) %></p>
                   </div>
                </div>
                <button class="geometry_light_button" onClick="copy_clipboard_WKT(event, '<%= pointe.point_attendu.x() %> <%= pointe.point_attendu.y() %>', 'expected_point_geometry_copy_ok_<%= pointe.id %>', 'expected_point_geometry_copy_nok_<%= pointe.id %>')" title="<%= l(:geometry_clic_button_copy_to_clipboard) %>">
                  <%= image_tag("copy_clipboard.png", :plugin => "geometry_status", :title => l(:geometry_clic_button_copy_to_clipboard), :align => "center") %>
                </button>
                <%= l(:geometry_expected_point) %>
                <br/>
              <% end %>
              <% if pointe.line_attendu != nil && pointe.line_attendu.as_text().length != 0 %>
                <!-- WKT of the expected line -->
                <div class="bubble" id="expected_line_geometry_copy_ok_<%= pointe.id %>" style="display:none;">
                  <div class="bubble-text">
                    <p><%= l(:text_copied_ok) %></p>
                   </div>
                </div>
                <div class="bubblenok" id="expected_line_geometry_copy_nok_<%= pointe.id %>" style="display:none">
                  <div class="bubblenok-text">
                    <p><%= l(:text_copied_nok) %></p>
                   </div>
                </div>
                <button class="geometry_light_button" onClick="copy_clipboard_WKT(event, '<%= pointe.line_attendu.as_text() %>', 'expected_line_geometry_copy_ok_<%= pointe.id %>', 'expected_line_geometry_copy_nok_<%= pointe.id %>')" title="<%= l(:geometry_clic_button_copy_wkt_to_clipboard) %>">
                  <%= image_tag("copy_clipboard_wkt.png", :plugin => "geometry_status", :title => l(:geometry_clic_button_copy_wkt_to_clipboard), :align => "center") %>
                </button>
                <%= l(:geometry_expected_line) %>
                <br/>
              <% end %>
              <% if pointe.polygon_attendu != nil && pointe.polygon_attendu.as_text().length != 0 %>
                <!-- WKT of the expected polygon -->
                <div class="bubble" id="expected_polygon_geometry_copy_ok_<%= pointe.id %>" style="display:none;">
                  <div class="bubble-text">
                    <p><%= l(:text_copied_ok) %></p>
                   </div>
                </div>
                <div class="bubblenok" id="expected_polygon_geometry_copy_nok_<%= pointe.id %>" style="display:none">
                  <div class="bubblenok-text">
                    <p><%= l(:text_copied_nok) %></p>
                   </div>
                </div>
                <button class="geometry_light_button" onClick="copy_clipboard_WKT(event, '<%= pointe.polygon_attendu.as_text() %>', 'expected_polygon_geometry_copy_ok_<%= pointe.id %>', 'expected_polygon_geometry_copy_nok_<%= pointe.id %>')" title="<%= l(:geometry_clic_button_copy_wkt_to_clipboard) %>">
                  <%= image_tag("copy_clipboard_wkt.png", :plugin => "geometry_status", :title => l(:geometry_clic_button_copy_wkt_to_clipboard), :align => "center") %>
                </button>
                <%= l(:geometry_expected_polygon) %>
                <br/>
              <% end %>
            </div>
            <div class="geometry_pointe_column2">
              <div style="display:<% if pointe.state == nil %>none<% else %>block<% end %>">
                <!-- State -->
                <legend><strong><%= l(:geometry_legend_state) %></strong></legend>
                <%= form.select(:statut, options_for_select([[l(:geometry_state_new), 1],
                                                             [l(:geometry_state_to_validated), 10],
                                                             [l(:geometry_state_resolved), 100],
                                                            ],
                                                            pointe.state)) %>
              </div>
            </div>
            <div class="geometry_pointe_column3">
              <div style="display:<% if pointe.state == nil %>none<% else %>block<% end %>">
                <!-- Comment -->
                <legend><strong><%= l(:geometry_legend_comment)%></strong></legend>
                <%= form.text_area :comment, rows: 5, cols: 50, class: 'geometry_comment', value: pointe.comment %>
              </div>
            </div>
            <div class="geometry_pointe_column4">
              <div style="display:<% if pointe.state == nil %>none<% else %>block<% end %>" class="geometry_pointe_vcenter">
                <div class="bubble" id="geometry_submit_ok_<%= pointe.id %>" style="display:none;">
                  <div class="bubble-text">
                    <p><%= l(:geometry_save_ok)%></p>
                   </div>
                </div>
                <div class="bubblenok" id="geometry_submit_failed_<%= pointe.id %>" style="display:none">
                  <div class="bubblenok-text">
                    <p><%= l(:geometry_save_failed) %></p>
                   </div>
                </div>
                <button name="commit" class="geometry_light_button" id="geometry_submit_<%= pointe.id %>" type="submit" onClick="onFormModifyPointe(event, this.form)">
                  <%= image_tag("submit.png", :plugin => "geometry_status", :title => l(:submit_statut_comment), :align => "center") %>
                </button>
              </div>
            </div>
          </fieldset>

Le code du javascript :

function onFormModifyPointe(evt, formData)
{
    // Very important!
    // If not these lines, clicking on button after moving it into a form will send the form and
    // force the page to be reloaded.
    evt = evt || window.event;

    var data = { "pointes" : {"state" : formData.pointes_state.value, "comment" : formData.pointes_comment.value } }
    var bRet = false;

    $.ajax({
        type: 'PUT',
        url: '../geometry/' + formData.pointes_id.value,
        contentType: 'application/json;charset=UTF-8',
        data: JSON.stringify(data), // access in body
        async: false, // Obligatoire pour exécuter evt.preventDefault() en cas d'échec AVANT le retour de la fonction
    }).done(function ()
    {
        document.getElementById("geometry_submit_ok_" + formData.pointes_id.value).style.display = "block";
        setTimeout(() =>
        {
          document.getElementById("geometry_submit_ok_" + formData.pointes_id.value).style.display = "none";
        }, 2000);
        bRet = true;
    }).fail(function (msg)
    {
        evt.preventDefault(); // Si échoue, ne pas faire le traitement sinon Redmine va considérer la FT comme étant sauvée même si la requête échoue (POST au lieu de PUT)
        document.getElementById("geometry_submit_failed_" + formData.pointes_id.value).style.display = "block";
        setTimeout(() =>
        {
          document.getElementById("geometry_submit_failed_" + formData.pointes_id.value).style.display = "none";
        }, 10000);
    }).always(function (msg)
    {
    });

    return bRet;
}

J'ai du d'ailleurs dû ajouter une route pour permettre au formulaire de faire un update en POST (alors qu'avant seul le PUT pour l'update était implémenté).
Du coup, sur la même fonction je peux faire un PUT ou un POST pour mettre à jour. Visiblement les formulaires ne se mettent à jour qu'en mode POST.

$ID = 'met la valeur que doit comporter le ID';
$nom = 'met la valeur que doit comporter le nom';
$req = $bdd->prepare("INSERT INTO az Kodi nox VALUES(:ID, :nom)");
$req -> bindvalue(':ID', $ID, PDO::PARAM_INT);
$req -> bindvalue(':nom', $nom, PDO::PARAM_STR);
$req -> execute();