Underscore: Modèles cassés en 1.7

Créé le 8 nov. 2014  ·  5Commentaires  ·  Source: jashkenas/underscore

Il semble que l'API pour les modèles ait changé. Je ne sais pas si cela est considéré comme une fonctionnalité, mais je considérerais cela comme un bogue car cela casse tout ce que j'ai jamais fait avec les modèles de soulignement.

Le problème est que je ne peux plus transmettre de données et récupérer le modèle rendu. Il renvoie maintenant une fonction au lieu d'une chaîne.

invalid

Commentaire le plus utile

Et je pensais que s'il y avait une bibliothèque js je n'aurais pas de problèmes de compatibilité serait souligné.... et dans une version mineure !!! il n'y a aucune excuse pour cela... :(

Tous les 5 commentaires

Selon le journal des modifications 1.7 : "Les modèles de soulignement n'acceptent plus un objet de données initial. _.template renvoie toujours une fonction maintenant". C'est donc le changement sans rétrocompatibilité.

Droit. N'est-ce pas une idée terrible? Que se passe-t-il si quelqu'un a un modèle côté serveur avec node.js et qu'il met à jour sans le vouloir le soulignement 1.7 ? Ils sont foutus. Pourquoi ne pas simplement vérifier si un objet est un objet de paramètres ou un objet de données en fonction des clés de l'objet ?

Voir https://github.com/jashkenas/underscore/issues/1736 pour le raisonnement, et https://github.com/jashkenas/underscore/issues/1805 pour les arguments que le soulignement devrait suivre semver.

Dans tous les cas, vous pouvez mettre à jour votre propre code en _.template(templateStr)(data) .

Il ne s'agissait pas de savoir comment contourner le problème. Je suis en train d'embrasser le changement. Sans tenir compte de tout le code que cela casse, du point de vue des performances et de l'API, cela a du sens. Ce n'était qu'une rude surprise de mettre à jour mon serveur de nœuds et de voir mes modèles d'e-mails cesser de fonctionner.

Merci d'avoir signalé les autres sujets de discussion. J'ai aimé lire les discussions.

Et je pensais que s'il y avait une bibliothèque js je n'aurais pas de problèmes de compatibilité serait souligné.... et dans une version mineure !!! il n'y a aucune excuse pour cela... :(

Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

jdalton picture jdalton  ·  6Commentaires

umarfarooq125 picture umarfarooq125  ·  8Commentaires

danilopolani picture danilopolani  ·  5Commentaires

marcalj picture marcalj  ·  5Commentaires

xiaoliwang picture xiaoliwang  ·  3Commentaires