Replies

• allan Posts: 61,972Questions: 1Answers: 10,160 Site admin

  May 16

  Given the aforementioned configuration (fields -> options) that links each value to a label, shouldn't the 'name' field in the response be entirely superfluous?

  Kind of. You are correct that it would be possible to look up the label on the client-side from the existing list of options, but Editor doesn't do that. Instead there is an assumption that the response from the server will be the full data for the row, in exactly the same format as when the table was initially loaded. You can see that in the example here.

  The reason Editor doesn't do the look up on the client-side is that I figured the server-side would run the same "get" code for the result of the edit as it would for the initial load, so there is no need for additional work at the client-side (which is exactly what the server-side libraries I publish for Editor do). There is also the small possibility that the label might have changed at the server-side between the updates, so this would make sure that the latest one is returned.

  Allan
• gravsten Posts: 17Questions: 5Answers: 0

  May 16

  Thanks for the detailed explanation.
  While the Editor's behaviour w.r.t. select options differs from regular HTML, it certainly makes sense.

  As a suggestion for a future version, you may consider looking up the label on the client-side from the existing list of options if the 'name' field is missing in the server response.

  Which is what I implemented through events:

      }).on('preSubmit', function (e, data, action) {
        var id, e;
        for (id in data.data) {
          e = data.data[id];
          if (e.hasOwnProperty('title'))
            e.title = e.title.id;
        }
      }).on('postSubmit', function (e, data, action) {
        var id, e, values = ["","Mr.","Mrs."];
        for (id in data.data) {
          e = data.data[id];
          if (e.hasOwnProperty('title'))
            e.title = { id: e.title, name: values[e.title] };
        }
      }),
  

  This way my server only needs to return the option's 'id' field, not having to load a full 150K of translations (among them the 'name' field in the correct language).

  Best regards,
  Guilhem.