@mbrodala रिपोर्ट के लिए बहुत-बहुत धन्यवाद, उन कोडपेन की बहुत सराहना की जाती है!

@mbrodala कोडपेन्स के लिए धन्यवाद।
मुझे आश्चर्य है कि क्या यहां कोई गलतफहमी हुई है।

643 और #664 एक बग को ठीक करें जिसकी मैंने #617 में रिपोर्ट की थी, जिसे इस परीक्षण द्वारा दर्शाया गया है, जो #643 के साथ है:

  describe('when parsing a template with tags specified followed by the same template with different tags specified', function() {
     it('returns different tokens for the latter parse', function() {
       var template = "(foo)[bar]";
       var parsedWithParens = Mustache.parse(template, ['(', ')']);
       var parsedWithBrackets = Mustache.parse(template, ['[', ']']);
       assert.notDeepEqual(parsedWithBrackets, parsedWithParens);
     });
   });

parse फ़ंक्शन कैश कुंजी के रूप में केवल template का उपयोग करके कैशिंग कर रहा था, ताकि अगली बार उस टेम्पलेट को पार्स करने के लिए parse का उपयोग किया जाए, यह बिल्कुल वही टोकन लौटाएगा, यहां तक ​​कि यदि निर्दिष्ट tags अलग हैं।

tags एक वैकल्पिक पैरामीटर है, और जब इसे छोड़ दिया जाता है, तो यह mustache.tags पर वापस आ जाता है, जो कि डिफ़ॉल्ट रूप से ['{{', '}}'] होता है। फ़ॉल बैक mustache.tags का उपयोग कैश कुंजी के भाग के रूप में किया जाता है।

मुझे लगता है कि मुझे पता है कि बग फिक्स और अपेक्षाओं के संबंध में क्या हो रहा है, और मैं इसके माध्यम से चलने की कोशिश करूंगा, और मैं उदाहरण के रूप में कोडपेन का उपयोग करूंगा।

v2.3.0

Mustache.parse(template, ['[[', ']]']);

2.3.0 में, यह मूंछों को टैग के रूप में ['[[', ']]'] का उपयोग करके $ template को पार्स करने का निर्देश देता है। मूंछें ऐसा करती हैं और सही परिणाम देती हैं, लेकिन केवल template का उपयोग करके कॉल को कैश करती हैं। [email protected] की पंक्तियाँ 447-450 देखें:

    if (tokens == null)
       tokens = cache[template] = parseTemplate(template, tags);

कोडपेन में अगली कॉल है:

var output = Mustache.render(
  template,
...

render एक tags पैरामीटर नहीं लेता है, इसलिए एक को parse पर पास नहीं करता है, इसलिए जब render को कॉल किया जाता है, तो parse mustache.tags उपयोग करता है render कॉल किया जाता है, तो यह प्रभावी रूप से parse कह रहा है, "कृपया template पार्स करें और ['{{', '}}'] को tags के रूप में उपयोग करें। parse वास्तव में गलत काम करता है और tags और mustache.tags दोनों को पूरी तरह से अनदेखा करते हुए कैश लुकअप करता है। यह [['[', ']']] के साथ पार्स किए गए टेम्पलेट के परिणाम को वापस करने के लिए होता है , लेकिन केवल इसलिए कि उस template के लिए पूरे कार्यक्रम में parse ['[[', ']']] के साथ की गई थी। tags ।

वी2.3.1

Mustache.parse(template, ['[[', ']]']);

पार्स परिणाम template और tags दोनों का उपयोग करके कैश किया जाता है, जो कैशकी के रूप में ['[[', ']]'] है।

अगली कॉल:

var output = Mustache.render(
  template,
...

render parse $ पर कॉल करता है, template पास करता है लेकिन tags को छोड़ देता है। parse इसलिए tags वापस गिरकर mustache.tags हो गया है, जो डिफ़ॉल्ट ['{{', '}}'] बना हुआ है। parse template ['{{', '}}'] parse template और टैग के उस संयोजन के साथ। इसलिए यह template का उपयोग करके ['{{', '}}'] का विश्लेषण करता है।

मेरा मानना ​​है कि v2.3.1 सही व्यवहार प्रदर्शित करता है। अगर हमें https://codepen.io/mbrodala/pen/QBJoOx में कोडपेन को थोड़ा बदलना है और इसे v2.3.0 के विरुद्ध चलाना है:

var template = "[[item.title]] [[item.value]]";
Mustache.parse(template, ['[[', ']]']);
var output = Mustache.render(
  template,
  {
    item: {
      title: "TEST",
      value: 1
    }
  }
);
alert(output);

आउटपुट [[item.title]] [[item.value]] है, जो अपेक्षित नहीं है।

मैं देख सकता हूं कि https://codepen.io/mbrodala/pen/NBEJjX में व्यवहार कैसे आश्चर्यजनक हो सकता है, क्योंकि Mustache.parse और Mustache.render कॉल एक दूसरे के ठीक बगल में हैं और एक नहीं हो सकता है यह भी महसूस करें कि Mustache.parse भी tags तर्क लेता है। ( Mustache.parse यहां तक ​​कि tags तर्क भी क्यों लेता है? यह mustache.js में कहीं भी उपयोग नहीं किया जाता है - parse केवल mustache.tags के लिए आंतरिक रूप से डिफ़ॉल्ट होता है। ..)

यदि व्यवहार में परिवर्तन वास्तव में बगफिक्स रिलीज की अपेक्षाओं को धता बताता है, तो मुझे बिल्कुल यकीन नहीं है कि क्या करना है। एक संभावना यह है कि #664 के साथ एक और बगफिक्स संस्करण जारी किया जाए, जो वास्तव में सभी कैशिंग व्यवहार को हटा देता है (यह देखते हुए कि #643 में, सभी कैश लुकअप छूट जाएंगे)। फिर हम #664 को अगले प्रमुख संशोधन में वापस रख सकते हैं। एक और संभावना है कि बगफिक्स रिलीज में सभी कैशिंग को हटा दिया जाए (गैर-कार्यात्मक कैशिंग के साथ mustache.js जारी करने के विपरीत), और फिर सभी कैशिंग को अगले प्रमुख संशोधन में वापस रखा जाए। पूर्व विकल्प में शायद कम जोखिम होता है (कम से कम कोड परिवर्तन की मात्रा) लेकिन बाद वाला विकल्प शायद अधिक "सही" है। @phillipj विचार?

विस्तृत शोध के लिए बहुत बहुत धन्यवाद जो मेरे पीओवी से पूरी तरह से समझ में आता है।

मुझे परिवर्तन के बारे में बिल्कुल भी कोई आपत्ति नहीं होगी, लेकिन यह देखते हुए कि कैश हिट सुनिश्चित करने के लिए tags से Mustache.render() पास करना असंभव है और Mustache.parse() को template कैश करने के लिए विज्ञापित किया गया है tags का कोई उल्लेख नहीं है) मुझे आश्चर्य है कि क्या इसे वास्तव में वापस किया जाना चाहिए।

यदि हम मानते हैं कि कोई Mustache.parse को tags के कस्टम सेट के साथ कॉल करता है, तो हम यह भी मान सकते हैं कि template इन सीमांककों का उपयोग करता है (BTW, "टैग" बनाम "सीमांकक" होना चाहिए भी साफ हो गया)। उसके बाद हम यह मान सकते हैं कि Mustache.render पर कॉल काम करने की उम्मीद है, भले ही दिए गए template को कैसे और कैसे कैश किया गया हो और अगर ऐसा है तो इसे कैसे संकलित किया गया। यदि कस्टम tags का उपयोग किया जाता है तो अभी इसकी गारंटी नहीं है।

@mbrodala हाँ यह समझ में आता है, हालांकि Mustache.parse(template, ['[[', ']]']); के बाद Mustache.parse(template, ['((', '))']); बिल्कुल वही परिणाम देना अभी भी अप्रत्याशित होगा।

यहां एक स्ट्रॉ-मैन समाधान/समझौता है ("स्ट्रॉ-मैन" क्योंकि मुझे यह पसंद नहीं है लेकिन यह दिमागी तूफान के लायक है)। हमारे पास अकेले $# template 3$#$ और टैग के साथ template दोनों के खिलाफ parse कैश हो सकता है। जब parse को tags निर्दिष्ट के साथ कॉल किया जाता है, तो यह template और tags के खिलाफ एक लुकअप करता है। जब हम render पर कॉल करते हैं, जो parse बिना tags के कॉल करता है, तो हम अकेले template के खिलाफ एक लुकअप करते हैं। विचार?

ध्वनि बंद है और मूल रूप से है, लेकिन फिक्स को बरकरार रखते हुए इस मुद्दे को ठीक कर देगा। मेरे पीओवी से ठीक है।

@mbrodala मुख्य मुद्दा है कि आप tags को render में पास नहीं कर सकते हैं? हम केवल tags पैरामीटर को render में जोड़ सकते हैं।

@petrkoutnysw क्या यह मोटे तौर पर वह समस्या है जिसका आप अनुभव कर रहे हैं?

यह कम से कम parse() और render() के बीच एक असंगति है। हम parse() का उपयोग भी नहीं करेंगे यदि हम वास्तव में render() पर कस्टम टैग पास कर सकते हैं। और अब उचित कैशिंग के साथ यह और अधिक स्पष्ट हो जाता है।

+1 बहुत सारे भ्रम को खत्म करने के लिए रेंडर() में टैग पैरामीटर जोड़ने के लिए - हमें इस बदलाव से भी थोड़ा सा मिला और लिंक बी/डब्ल्यू पार्स और रेंडर हमेशा आवश्यकता से थोड़ा अधिक जादू की तरह लग रहा था।

ठीक है, तो हम बग फिक्स रिलीज में कैशिंग को कैसे अक्षम करते हैं, तत्काल समस्या को ठीक करने और सिमेंटिक वर्जनिंग का अनुपालन करने के लिए, और अगली प्रमुख रिलीज में render विधि में इसे और tags को फिर से प्रस्तुत करते हैं? (फिर से, मैं प्रस्तावित स्ट्रॉ-मैन समाधान का बहुत बड़ा प्रशंसक नहीं हूं।)

उस संपूर्ण पूर्वाभ्यास के लिए बहुत-बहुत धन्यवाद @ रेमंड-लैम!

मैं सुझाए गए बग फिक्स रिलीज की ओर झुक रहा हूं, मुख्य रूप से सेवर चिंताओं और परियोजनाओं के कारण जहां व्यवहार में यह परिवर्तन अप्रत्याशित है और इसलिए जंगली में परियोजनाओं में विनाश का कारण बन सकता है।

नियोजित अगली प्रमुख रिलीज़ में कैशिंग व्यवहार को फिर से प्रस्तुत करते हुए, हम रिलीज़ नोट्स में माइग्रेशन निर्देश शामिल कर सकते हैं।

@phillipj मैंने पुल अनुरोध #670 जारी किया है जो #643 और #664 वापस आ गया है। सभी को एक साथ कैशिंग अक्षम करने के बजाय, जोखिम कम करने के लिए, बस v2.3.0 व्यवहार पर वापस जाना (एक बगफिक्स रिलीज़ में) मूंछें v2.xx पर आश्रितों के लिए सबसे सुरक्षित लगता है मैं एक प्रमुख रिलीज में पुन: प्रस्तुत करने के लिए एक और पुल अनुरोध जारी करूंगा।

@phillipj #671 एक प्रमुख रिलीज़ पर प्रतीक्षा करने के लिए, कैशिंग फ़िक्सेस को फिर से प्रस्तुत करता है।

tags को `रेंडर करने के लिए जोड़ने के लिए समस्या #672 बनाई गई।

इसे देखने और इसे ठीक करने के लिए बहुत-बहुत धन्यवाद, आप लोग रॉक करें। मैं

इसके लिए @ रेमंड-लैम को सलाम! आपके लिए भी धन्यवाद, हमारे लिए यह जानना महत्वपूर्ण है कि जंगली में अप्रत्याशित परिवर्तन कब होते हैं।

v2.3.2 प्रकाशित किया गया है