Merge Request fermées ?
Hello Dimitry, je vois que les merges requests sont fermées, c'est une volontée particulière? je pensais proposer quelques modifications sur le javascript juste pour apporter des modernisations et partager un peu de savoir sur les avancées de javascript qui peuvent eviter des mauvaises surprises.
Tel que:
Commit message:
Notation littérale création d'objet et attribut pour accès.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Object_initializer
fichier: wp-organisme-de-formation/js/wpof.js
@@ -2038,14 +2038,13 @@ $(".quiz-enregistrer").click(function (e)
bouton = $(this);
parent_id = $(".id").attr('data-id');
quiz_id = $(this).closest('.quiz').attr('data-id');
- questions = Array();
+ const questions = [];
sortable = $(this).closest('.quiz').find("ul.quiz-sortable");
- sortable.children().each(function()
- {
- q = Object();
- q.type = $(this).attr('data-type');
- q.text = $(this).find("input[type='text']").val();
- questions.push(q);
+ sortable.children().each(function() {
+ questions.push({
+ type: $(this).attr('data-type'),
+ text: $(this).find("input[type='text']").val()
+ });
});
ps: je bosse a coopaname en dev, mais c'est à titre personne que je pensais contribuer un peu sur le js. ;)
Edited by Viala axel