Handlebars.js: गतिशील आंशिक ब्लॉक का समर्थन करें

को निर्मित 11 अग॰ 2017 · 11टिप्पणियाँ · स्रोत: handlebars-lang/handlebars.js

वर्तमान में, ब्लॉक सिंटैक्स के साथ एक गतिशील आंशिक को परिभाषित करने का कोई तरीका नहीं है, ताकि यदि उप अभिव्यक्ति एक अपरिभाषित आंशिक को हल करती है तो एक विफलता प्रदान की जा सकती है।

यह महत्वपूर्ण क्यों है?

जब हम एक गतिशील आंशिक ब्लॉक का उपयोग करते हैं, तो मॉडल-बिल्डिंग कोड को पता नहीं हो सकता है कि कौन से आंशिक पंजीकृत हैं। उदाहरण के लिए, हम उन नामों के साथ आंशिक पंजीकरण कर सकते हैं जो त्रुटि कोड से मेल खाते हैं, ताकि हम कुछ मामलों में एक त्रुटि-विशिष्ट टेम्पलेट प्रस्तुत कर सकें, जो डिफ़ॉल्ट टेम्पलेट पर वापस आते हैं। इसलिए, जब डायनेमिक आंशिक पंजीकृत आंशिक का समाधान नहीं करता है, तो उपयोग करने के लिए एक डिफ़ॉल्ट टेम्पलेट की आपूर्ति करना सुविधाजनक होगा।

मैंने पाया कि यदि closeBlock helperName वैकल्पिक है तो इस व्यवहार का समर्थन करना काफी तुच्छ है:

diff --git a/lib/handlebars/compiler/helpers.js b/lib/handlebars/compiler/helpers.js
index 3c1a38f..f6295d8 100644
--- a/lib/handlebars/compiler/helpers.js
+++ b/lib/handlebars/compiler/helpers.js
@@ -1,9 +1,11 @@
 import Exception from '../exception';

 function validateClose(open, close) {
-  close = close.path ? close.path.original : close;
+  if (close.hasOwnProperty('path')) {
+    close = close.path ? close.path.original : close.path;
+  }

-  if (open.path.original !== close) {
+  if (close && open.path.original !== close) {
     let errorNode = {loc: open.path.loc};

     throw new Exception(open.path.original + " doesn't match " + close, errorNode);
diff --git a/spec/partials.js b/spec/partials.js
index 266837d..7982677 100644
--- a/spec/partials.js
+++ b/spec/partials.js
@@ -19,6 +19,20 @@ describe('partials', function() {
     shouldCompileToWithPartials(string, [hash, helpers, {dude: partial}], true, 'Dudes: Yehuda (http://yehuda) Alan (http://alan) ');
     shouldCompileToWithPartials(string, [hash, helpers, {dude: partial},, false], true, 'Dudes: Yehuda (http://yehuda) Alan (http://alan) ');
   });
+
+  it('dynamic partials with failover', function() {
+    var string = 'Dudes: {{#dudes}}{{#> (partial)}}Anonymous {{/}}{{/dudes}}';
+    var partial = '{{name}} ({{url}}) ';
+    var hash = {dudes: [{name: 'Yehuda', url: 'http://yehuda'}, {name: 'Alan', url: 'http://alan'}]};
+    var helpers = {
+      partial: function() {
+        return 'missing';
+      }
+    };
+    shouldCompileToWithPartials(string, [hash, helpers, {dude: partial}], true, 'Dudes: Anonymous Anonymous ');
+    shouldCompileToWithPartials(string, [hash, helpers, {dude: partial},, false], true, 'Dudes: Anonymous Anonymous ');
+  });
+
   it('failing dynamic partials', function() {
     var string = 'Dudes: {{#dudes}}{{> (partial)}}{{/dudes}}';
     var partial = '{{name}} ({{url}}) ';
diff --git a/src/handlebars.yy b/src/handlebars.yy
index ce06498..1df03cf 100644
--- a/src/handlebars.yy
+++ b/src/handlebars.yy
@@ -79,7 +79,7 @@ inverseChain
   ;

 closeBlock
-  : OPEN_ENDBLOCK helperName CLOSE -> {path: $2, strip: yy.stripFlags($1, $3)}
+  : OPEN_ENDBLOCK helperName? CLOSE -> {path: $2, strip: yy.stripFlags($1, $3)}
   ;

 mustache

बेशक, यह समाधान किसी भी ब्लॉक को {{/}} साथ बंद करने की अनुमति देता है, जो एक नियमित अभ्यास के रूप में अनुचित होगा, लेकिन कुछ भी नुकसान नहीं पहुंचाता है। validateClose में केवल खाली बंद होने की अनुमति देने के लिए एक अतिरिक्त चेक शामिल हो सकता है जब open.type === 'SubExpression' , ताकि केवल एक गतिशील आंशिक ब्लॉक को खाली बंद होने की अनुमति हो (मुझे लगता है कि यह एकमात्र मामला है जब खुली इच्छा SubExpression )।

एक अन्य विकल्प partialName को closeBlock :

diff --git a/spec/partials.js b/spec/partials.js
index 266837d..4800d3f 100644
--- a/spec/partials.js
+++ b/spec/partials.js
@@ -19,6 +19,20 @@ describe('partials', function() {
     shouldCompileToWithPartials(string, [hash, helpers, {dude: partial}], true, 'Dudes: Yehuda (http://yehuda) Alan (http://alan) ');
     shouldCompileToWithPartials(string, [hash, helpers, {dude: partial},, false], true, 'Dudes: Yehuda (http://yehuda) Alan (http://alan) ');
   });
+
+  it('dynamic partials with failover', function() {
+    var string = 'Dudes: {{#dudes}}{{#> (partial)}}Anonymous {{/(partial)}}{{/dudes}}';
+    var partial = '{{name}} ({{url}}) ';
+    var hash = {dudes: [{name: 'Yehuda', url: 'http://yehuda'}, {name: 'Alan', url: 'http://alan'}]};
+    var helpers = {
+      partial: function() {
+        return 'missing';
+      }
+    };
+    shouldCompileToWithPartials(string, [hash, helpers, {dude: partial}], true, 'Dudes: Anonymous Anonymous ');
+    shouldCompileToWithPartials(string, [hash, helpers, {dude: partial},, false], true, 'Dudes: Anonymous Anonymous ');
+  });
+
   it('failing dynamic partials', function() {
     var string = 'Dudes: {{#dudes}}{{> (partial)}}{{/dudes}}';
     var partial = '{{name}} ({{url}}) ';
diff --git a/src/handlebars.yy b/src/handlebars.yy
index ce06498..2d63945 100644
--- a/src/handlebars.yy
+++ b/src/handlebars.yy
@@ -79,7 +79,7 @@ inverseChain
   ;

 closeBlock
-  : OPEN_ENDBLOCK helperName CLOSE -> {path: $2, strip: yy.stripFlags($1, $3)}
+  : OPEN_ENDBLOCK partialName CLOSE -> {path: $2, strip: yy.stripFlags($1, $3)}
   ;

 mustache

मुझे नहीं लगता कि यह व्यवहार में एक बहुत अच्छा समाधान होगा क्योंकि गतिशील आंशिक अभिव्यक्ति शायद ही कभी इतनी छोटी होती है, इसलिए openPartialBlock और closeBlock में उप अभिव्यक्ति का मिलान करना काफी कठिन होगा।

मैं एक उप अभिव्यक्ति वाले openPartialBlock को लेबल करने का एक तरीका खोजना चाहता था, लेकिन ऐसा करने का एक तरीका नहीं समझ सका जो मौजूदा आंशिक वाक्यविन्यास से स्पष्ट रूप से असंबद्ध है। मुझे अन्य वाक्यात्मक सुझावों को सुनना अच्छा लगेगा, और अगर वाक्य रचना पर आम सहमति हो तो मुझे पीआर जमा करने में खुशी होगी।

feature

स्रोत

jstewmon

👍4

सबसे उपयोगी टिप्पणी

इसके अलावा, बस एक SO थ्रेड में आया जो सही ढंग से नोट करता है कि आप गतिशील आंशिक का उपयोग ब्लॉक-स्तरीय सहायकों के रूप में _today_ इस 'lil हैक' के साथ कर सकते हैं:

{{#>( lookup . 'intendedTemplate' )}}
  No template matched for "{{intendedTemplate}}"
 {{/undefined}}

अपने क्लोजिंग टैग के रूप में {{/undefined}} का उपयोग करना, कम से कम कहने के लिए हैकी लगता है। फिर भी - भले ही इसमें @nknapp के सुझाए गए सिंटैक्स की सुंदरता का अभाव है - यह वास्तव में अपेक्षित रूप से कार्य करता है।

rafegoldberg 30 जन॰ 2018

❤7 👍3 🎉1

सभी 11 टिप्पणियाँ

सिंटैक्स के दृष्टिकोण से, डेकोरेटर सिंटैक्स जिसे इनलाइन पार्टिकल्स का समर्थन करने के लिए पेश किया गया था, यहाँ एक अच्छा फिट हो सकता है। कुछ इस तरह:

{{#*dynamic (partialName)}}
Fallback content
{{/dynamic}}

lawnsea 12 अग॰ 2017

👍3

यह जानना दिलचस्प होगा कि क्या इसे डेकोरेटर के रूप में लागू किया जा सकता है (यह मानते हुए कि #1372 तय किया जाएगा)।

संपादित करें: नहीं, मेरी टिप्पणी गूंगा थी। यह कोई डेकोरेटर नहीं है। कुछ भी हो, तो वह सहायक होगा, लेकिन सहायक आसानी से आंशिक नहीं कह सकते।
यदि आप Handlebars.registeryDecorator('dynamic',...) पहले करते हैं, तो सिंटैक्स {{#*dynamic ...}} पहले ही लिया जा चुका है।

nknapp 14 अग॰ 2017

संपादित करें: इस बारे में सोचने के बाद, निम्नलिखित शायद एक बहुत ही खराब समाधान है। मुझे साइड-इफेक्ट्स के बारे में निश्चित नहीं है और यह बहुत सारे एपीआई का उपयोग करता है जिसे मैं सार्वजनिक नहीं कहूंगा ...

इस तरह कुछ "सभी" लापता आंशिक के लिए एक डिफ़ॉल्ट स्ट्रिंग को लागू करने के लिए इस्तेमाल किया जा सकता है।

const Handlebars = require('handlebars')

Handlebars.registerDecorator('onMissingPartial', function(program, props, container, decoratorContext) {
  const invokePartialWrapper = container.invokePartial
  container.invokePartial = function (partial, context, options) {
    if (props.partials[options.name]) {
      return invokePartialWrapper.apply(this, Array.prototype.slice.call(arguments))
    }
    return decoratorContext.fn(Object.assign({partialName: options.name, context}))
  }
});

var template = Handlebars.compile(`
{{#*inline "inlinePartial"}}
inlinePartial
{{/inline}}

{{#*onMissingPartial}}
Partial "{{partialName}}" could not be found
{{/onMissingPartial}}

{{>inlinePartial}}
{{>unknownPartial}}
`)

console.log("output", template({name: 'myPartial2'}))

यह दिखाता है

Partial "unknownPartial" could not be found

यह {{> (name) }} साथ काम नहीं करता है, लेकिन मुझे लगता है कि यह #1372 के समान एक बग हो सकता है।

मैं नई भाषा सुविधाओं के बिना एक रास्ता खोजने की कोशिश कर रहा हूं, क्योंकि हमें पहले एक युक्ति की आवश्यकता है (#1277), और इस समय, कोई भी एक लिख रहा प्रतीत नहीं होता है।

nknapp 15 अग॰ 2017

मैं इसे लागू करूंगा, मुझे लगता है कि मैं मूल रूप से सुझाए गए {{/}} क्लोजिंग ब्लॉक के लिए जाऊंगा।

nknapp 15 अग॰ 2017

यदि आप Handlebars.registeryDecorator('dynamic',...) पहले करते हैं, तो सिंटैक्स {{#*dynamic ...}} पहले ही लिया जा चुका है।

क्या आप कह रहे हैं कि ऐसा कोड है जो पहले से ही dynamic डेकोरेटर पंजीकृत करता है, या आप कह रहे हैं कि कोड इसे पंजीकृत कर सकता है?

lawnsea 15 अग॰ 2017

आप इसे रजिस्टर कर सकते हैं। लेकिन {{#*...}} अर्थ है "डेकोरेटर लागू करें", इसलिए {{#* dyamic}} का अर्थ "डेकोरेटर" भी होगा। {{#*inline}} एक डेकोरेटर इस संदर्भ के लिए एक आंशिक जोड़ता है। लेकिन डायनेमिक ब्लॉक-पार्टिकल को कॉल करना डेकोरेटर की तरह नहीं है।

के बाद से {{helper}} हो जाता है {{#block-helper}}...{{/block-helper}} ब्लॉकों के लिए और {{>partial}} हो जाता है {{#>block-partial}}...{{/block-partial}} , और {{> (dynamic-partial}} वाक्य रचना पहले से मौजूद है, मुझे लगता है कि {{#> (dynamic-block) }} तार्किक विकल्प है। केवल गायब चीज अंत टैग है और {{/}} तार्किक लगता है और कुछ भी नहीं तोड़ता है। (जहाँ तक मैं देख सकता हूँ)।

nknapp 15 अग॰ 2017

👍3

डायनेमिक पार्टिकल्स के लिए फ़ेलओवर सामग्री ब्लॉक हैंडलबार्स के लिए एक _fantastic_ अतिरिक्त होगा। क्या इस पर कोई आंदोलन हुआ है? क्या ऐसा कुछ है जो मैं इसे आगे बढ़ाने में मदद कर सकता हूं? (इसके अलावा: एचबीएस के किसी भी/सभी सहयोगियों को उनके शानदार काम के लिए बहुत !)

rafegoldberg 30 जन॰ 2018

{{#>( lookup . 'intendedTemplate' )}}
  No template matched for "{{intendedTemplate}}"
 {{/undefined}}

rafegoldberg 30 जन॰ 2018

❤7 👍3 🎉1

सभी रचनात्मक प्रतिक्रिया के बाद पीआर के बिना इस पर रोक लगाने के लिए खेद है। मैं यह सुनिश्चित करने की कोशिश कर रहा था कि खाली समापन ब्लॉक अन्य आंशिक ब्लॉकों के लिए स्वीकार्य नहीं था, लेकिन मैं आज सब कुछ वापस चला गया और सोचता हूं कि यह अब # 1422 में काम कर रहा है।

jstewmon 31 जन॰ 2018

👍1

पीआर के लिए धन्यवाद, @jstewmon। मुझे लगता है कि अंतिम निर्णय @wycats द्वारा किया
यदि/जब यह मुख्य समस्या हल हो जाती है, तो मेरे पास कुछ टिप्पणियाँ हैं:

1) मैं {{#> (partial)}}failover content{{/}} जैसे टेम्पलेट के साथ एक परीक्षण देखना चाहता हूं जहां "आंशिक" वास्तव में मौजूदा आंशिक को हल करता है।
2) इस प्रतिबद्धता के साथ प्रदर्शन कैसे बदलता है?
3) यदि आप इस सुविधा को 4.1 में देखना चाहते हैं (यदि @wycats स्वीकृत है), तो आपको अपने पीआर को 4.x -branch पर इंगित करना चाहिए। मैं बाद में मास्टर में विलय का ध्यान रखूंगा।

nknapp 9 फ़र॰ 2018

#1422 में लागू किया गया लेकिन अभी तक विलय नहीं हुआ है

nknapp 5 अप्रैल 2020

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Handlebars.js: गतिशील आंशिक ब्लॉक का समर्थन करें

सबसे उपयोगी टिप्पणी

सभी 11 टिप्पणियाँ

संबंधित मुद्दों