prepare.js ➔ ... ➔ ??? - Code Metrics - Inspection of "Merge pull request #78 from AdFabConnect/bug_77" - AdFabConnect/abejs - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c2f34b...8075f4 )

by greg

created 2016-12-13 19:45 UTC

prepare.js ➔ ... ➔ ??? A

↳ Parent: prepare.js ➔ addAbeDataAttrForHtmlAttributes

Complexity

Conditions	3
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
c	0
b	0
f	0
nc	2
nop	1
dl	0
loc	17
rs	9.4285

import Handlebars from 'handlebars'

import {
  cmsData
  ,config
  ,cmsTemplates
} from '../../'

/**
 * THIS:
<span>{{abe type='text' key='text_visible'}}</span>

 * BECOME:
<span data-abe-text_visible="text_visible" >{{abe type='text' key='text_visible'}}</span>

 * @param {[type]} template [description]
 */
export function addAbeDataAttrForHtmlTag(template) {
  var match
  while (match = cmsData.regex.abePattern.exec(template)) {
    var getattr = cmsData.regex.getAttr(match, 'key').replace(/\./g, '-')
    template = template.replace(
      cmsData.regex.escapeTextToRegex(match[0], 'g'),
      ' data-abe-' + cmsData.regex.validDataAbe(getattr) + '="'  + getattr + '" ' + match[0]
    )
  }

  return template
}

export function addHasAbeAttr(text) {
  return text.replace('}}', ' has-abe=1}}')
}

export function getAbeAttributeData(match, text, htmlAttribute, abeTag) {
  var valueOfAttritube
  var key = cmsData.regex.getAttr(match, 'key')
  var getattr
  var res

  if (cmsData.regex.isSingleAbe(match, text)) {
    valueOfAttritube = key.replace(/\./g, '-')
    key = cmsData.regex.validDataAbe(valueOfAttritube)
    getattr = key.replace(/\./g, '-')
    res = ' data-abe-attr-' + valueOfAttritube + '="'  + htmlAttribute + '"' + ' data-abe-' + valueOfAttritube + '="'  + getattr + '"' + abeTag
  }else {
    var valueOfAttritube = key.split('.')

    var parentKey = valueOfAttritube.shift()
    valueOfAttritube = `${parentKey}[index].${valueOfAttritube[0]}`
    var valueOfAttritubeIndexed = valueOfAttritube.replace(/\[index\]/, '{{@index}}')
    key = cmsData.regex.validDataAbe(valueOfAttritube)
    getattr = key

    res = ` data-abe-attr-${valueOfAttritube}="${htmlAttribute}"  data-abe-${valueOfAttritube}="${getattr}"`
    + ` data-abe-attr-${valueOfAttritubeIndexed}="${htmlAttribute}" data-abe-${valueOfAttritubeIndexed}="${getattr}"${abeTag}`
  }

  return res
}

/**
 *
 * IF ABE TAG SINGLE (NOT ABE EACH STATEMENT)
 * 
 * THIS:
<img src="{{abe type='image' key='image_key' tab='default'}}" alt="">

 * BECOME:
<img data-abe-attr-image_key="src" data-abe-image_key="image_key" data-abe-attr-image_key="src"
data-abe-image_key="image_key" src="{{abe type='image' key='image_key' tab='default' has-abe=1 has-abe=1}}" alt="">

 *
 * IF ABE EACH TAG
 * THIS:
{{#each test}}
  <img src="{{abe type='image' key='test.img' desc='test_img' tab='default'}}" alt="">
{{/each}}

 * BECOME:
{{#each test}}
  <img data-abe-attr-test[index].img="src" data-abe-test[index].img="test[index].img" src="{{abe type='image' key='test.img' desc='test_img' tab='default' has-abe=1}}" alt="">
{{/each}}

 * @param {[type]} template [description]
 */
export function addAbeDataAttrForHtmlAttributes(template, key) {

  var text = template.replace(/<([A-Za-z]+)/g, '\nABE_SPLIT<$1')
  let abeTagIntoAttribute = text.match(cmsData.regex.abeAsAttributePattern)

  Array.prototype.forEach.call(abeTagIntoAttribute, (abeIntoTag) => {
    let matchAbeTag = /({{abe.*?[\s\S].*?}})/g.exec(abeIntoTag)

    if(matchAbeTag != null && matchAbeTag[1] != null) {
      var toReplace = cmsTemplates.prepare.getAbeAttributeData(matchAbeTag[1], text, (abeIntoTag.split('=')[0]).trim(), abeIntoTag)

      toReplace = toReplace.replace(
        cmsData.regex.escapeTextToRegex(matchAbeTag[1]),
        cmsTemplates.prepare.addHasAbeAttr(matchAbeTag[1])
      )

      text = text.replace(
        cmsData.regex.escapeTextToRegex(abeIntoTag),
        toReplace
      )
    }
  })
  text = text.replace(/\nABE_SPLIT</g, '<')

  return text
}

/**
 * Example:
 *
 *
 * THIS:
{{abe type='data' key='data_key' source='select title from article' display='title' editable='true' tab='default'}}

{{#each data_key}}
  {{title}}
{{/each}}

 *
 * BECOME THIS

{{abe type='data' key='data_key' source='select title from article' display='title' editable='true' tab='default'}}

{{#each data_key}}
  {{title}}
{{/each}}<!-- [[data_key]] %7B%7B%23each%20data_key%7D%7D%0A%09%7B%7Btitle%7D%7D%0A%7B%7B/each%7D%7D -->

 * @param {[type]} template [description]
 * @param {[type]} json     [description]
 */
export function addAbeSourceComment(template, json) {
  
  // Don't know what it does...
  if(typeof json.abe_source !== 'undefined' && json.abe_source !== null) {
    var keys = Object.keys(json.abe_source)
    
    for(var i in keys) {
var someObject;
for (var key in someObject) {
    if ( ! someObject.hasOwnProperty(key)) {
        continue; // Skip keys from the prototype.
    }

    doSomethingWith(key);
}
      var replaceEach = new RegExp(`<!-- \\[\\[${keys[i]}\\]\\][\\s\\S]*?-->`, 'g')
      template = template.replace(replaceEach, '')

      var patAttrSource = new RegExp(' ([A-Za-z0-9\-\_]+)=["|\'].*?({{' + keys[i] + '}}).*?["|\']', 'g')
      var patAttrSourceMatch = template.match(patAttrSource)

      if(patAttrSourceMatch != null) {
        let checkEscapedRegex = /["|'](.*?)["|']/
        let patAttrSourceInside = new RegExp('(\\S+)=["\']?((?:.(?!["\']?\\s+(?:\\S+)=|[>"\']))+.)["\']?({{' + keys[i] + '}}).*?["|\']', 'g')
        Array.prototype.forEach.call(patAttrSourceMatch, (pat) => {
          let patAttrSourceCheck = patAttrSourceInside.exec(pat)
          if(patAttrSourceCheck != null) {
            
            let checkEscaped = checkEscapedRegex.exec(patAttrSourceCheck[0])
            if(checkEscaped != null && checkEscaped.length > 0) {
              checkEscaped = escape(checkEscaped[1])
              template = template.replace(

                patAttrSourceCheck[0],
                ` data-abe-attr="${patAttrSourceCheck[1]}" data-abe-attr-escaped="${checkEscaped}" data-abe="${keys[i]}" ${patAttrSourceCheck[0]}`

              )
            }
          }
        })
      }

      var eachSource = new RegExp(`({{#each ${keys[i]}}[\\s\\S a-z]*?{{\/each}})`, 'g')
      var matches = template.match(eachSource)
      if(typeof matches !== 'undefined' && matches !== null) {
        Array.prototype.forEach.call(matches, (match) => {
          template = template.replace(match, `${match}<!-- [[${keys[i]}]] ${cmsTemplates.encodeAbeTagAsComment(match)} -->`)

        })
      }
    }
  }

  return template
}

/**
 * THIS:
<span>{{abe type='text' key='text_visible'}}</span>

 * BECOME:
<span><abe>{{abe type='text' key='text_visible'}}</abe></span>

 * @param {[type]} template [description]
 */
export function addAbeHtmlTagBetweenAbeTags(template) {
  var match
  while (match = cmsData.regex.abePattern.exec(template)) {
    template = template.replace(cmsData.regex.escapeTextToRegex(match[1], 'g'), '<abe>' + match[1].trim() + '</abe>')
  }

  return template
}

/**
 * THIS:
[index].

 * BECOME:
{{@index}}-

 *  @param  {[type]} template [description]
 * @return {[type]}          [description]
 */
export function replaceAbeEachIndex(template) {
  return template.replace(/\[index\]\./g, '{{@index}}-')
}

export function removeHiddenAbeTag(template) {
  return template.replace(/(\{\{abe.*visible=[\'|\"]false.*\}\})/g, '')
}

/**
 * Remove {{abe type=*}} from html if attribute visible="false"
 * @param  {[type]} template [description]
 * @return {[type]}          [description]
 */
export function removeHandlebarsRawFromHtml(template) {
  return template.replace(/\{\{\{\{\/?raw\}\}\}\}/g, '')
}

/**
 * split {{#each}}...{{/each}} into an array
 * 
 * @param  {[type]} template [description]
 * @return {[type]}          [description]
 */
export function splitEachBlocks(template) {
  var block
  var blocks = []

  while (block = cmsData.regex.blockPattern.exec(template)) {
    blocks.push(block[1])
  }

  return blocks
}

export function indexEachBlocks(template, onlyHtml) {
  // create an array of {{each}} blocks
  var blocks = cmsTemplates.prepare.splitEachBlocks(template)

  Array.prototype.forEach.call(blocks, (block) => {
    var key = block.match(/#each (.*)\}\}/)[1]
    var match

    if(!onlyHtml) {

      var voidData = {}
      voidData[key] = [{}]
      var blockCompiled = Handlebars.compile(block.replace(/{{abe (.*?)}}/g, '[[abe $1]]').replace(new RegExp(`\\.\\.\/${config.meta.name}`, 'g'), config.meta.name))
      var blockHtml = blockCompiled(voidData, {data: {intl: config.intlData}}).replace(/\[\[abe (.*?)\]\]/g, '{{abe $1}}')

      // je rajoute un data-abe-block avec index sur tous les tags html du bloc each
      var textEachWithIndex = block.replace(/(<(?![\/])[A-Za-z0-9!-]*)/g, '$1 data-abe-block="' + key + '{{@index}}"')

      // je remplace le block dans le texte par ça
      template = template.replace(block, textEachWithIndex + `<!-- [[${key}]] ${cmsTemplates.encodeAbeTagAsComment(blockHtml)} -->`)
    }

    // Pour chaque tag Abe
    while (match = cmsData.regex.abeTag.exec(block)) {
      // template = cmsTemplates.prepare.insertAbeEach(template, match, key, cmsData.regex.eachBlockPattern.lastIndex - block.length, onlyHtml)
      template = cmsTemplates.prepare.addAbeDictionnary(template, match[0], key)

    } 
  })

  return template
}

/**
 * split {{#each}}...{{/each}} into an array
 *
 * THIS:
  {{abe type='text' key='test.title' desc='test title' tab='default'}}

 * BECOME THIS:
  {{abe dictionnary='test' type='text' key='test.title' desc='test title' tab='default'}}

 * 
 * @param  {[type]} template [description]
 * @return {[type]}          [description]
 */
export function addAbeDictionnary(template, match, key) {
  if(cmsData.regex.isEachStatement(match)) return
if (a > 0)
    b = 42;

  if(cmsData.regex.isBlockAbe(match)){
    var abeDictionnary = match.replace(new RegExp('(key=[\'|"])' + key + '.', 'g'), '$1' + key + '[index].')
                               .replace(/\{\{abe/, '{{abe dictionnary=\'' + key + '\'')

    template = template.replace(match, abeDictionnary)
  }

  return template
}

1			import Handlebars from 'handlebars'
2
3			import {
4			cmsData
5			,config
6			,cmsTemplates
7			} from '../../'
8
9			/**
10			* THIS:
11			<span>{{abe type='text' key='text_visible'}}</span>
12
13			* BECOME:
14			<span data-abe-text_visible="text_visible" >{{abe type='text' key='text_visible'}}</span>
15
16			* @param {[type]} template [description]
17			*/
18			export function addAbeDataAttrForHtmlTag(template) {
19			var match
20			while (match = cmsData.regex.abePattern.exec(template)) {
21			var getattr = cmsData.regex.getAttr(match, 'key').replace(/\./g, '-')
22			template = template.replace(
23			cmsData.regex.escapeTextToRegex(match[0], 'g'),
24			' data-abe-' + cmsData.regex.validDataAbe(getattr) + '="' + getattr + '" ' + match[0]
25			)
26			}
27
28			return template
29			}
30
31			export function addHasAbeAttr(text) {
32			return text.replace('}}', ' has-abe=1}}')
33			}
34
35			export function getAbeAttributeData(match, text, htmlAttribute, abeTag) {
36			var valueOfAttritube
37			var key = cmsData.regex.getAttr(match, 'key')
38			var getattr
39			var res
40
41			if (cmsData.regex.isSingleAbe(match, text)) {
42			valueOfAttritube = key.replace(/\./g, '-')
43			key = cmsData.regex.validDataAbe(valueOfAttritube)
44			getattr = key.replace(/\./g, '-')
45			res = ' data-abe-attr-' + valueOfAttritube + '="' + htmlAttribute + '"' + ' data-abe-' + valueOfAttritube + '="' + getattr + '"' + abeTag
46			}else {
47			var valueOfAttritube = key.split('.')
			0 ignored issues – show Comprehensibility Naming Best Practice introduced 2016-12-13 17:38 UTC by Report Bug Copy Issue Report The variable `valueOfAttritube` already seems to be declared on line `36`. Consider using another variable name or omitting the `var` keyword. This check looks for variables that are declared in multiple lines. There may be several reasons for this. In the simplest case the variable name was reused by mistake. This may lead to very hard to locate bugs. If you want to reuse a variable for another purpose, consider declaring it at or near the top of your function and just assigning to it subsequently so it is always declared. Loading history...
48			var parentKey = valueOfAttritube.shift()
49			valueOfAttritube = `${parentKey}[index].${valueOfAttritube[0]}`
50			var valueOfAttritubeIndexed = valueOfAttritube.replace(/\[index\]/, '{{@index}}')
51			key = cmsData.regex.validDataAbe(valueOfAttritube)
52			getattr = key
53
54			res = ` data-abe-attr-${valueOfAttritube}="${htmlAttribute}" data-abe-${valueOfAttritube}="${getattr}"`
55			+ ` data-abe-attr-${valueOfAttritubeIndexed}="${htmlAttribute}" data-abe-${valueOfAttritubeIndexed}="${getattr}"${abeTag}`
56			}
57
58			return res
59			}
60
61			/**
62			*
63			* IF ABE TAG SINGLE (NOT ABE EACH STATEMENT)
64			*
65			* THIS:
66			<img src="{{abe type='image' key='image_key' tab='default'}}" alt="">
67
68			* BECOME:
69			<img data-abe-attr-image_key="src" data-abe-image_key="image_key" data-abe-attr-image_key="src"
70			data-abe-image_key="image_key" src="{{abe type='image' key='image_key' tab='default' has-abe=1 has-abe=1}}" alt="">
71
72			*
73			* IF ABE EACH TAG
74			* THIS:
75			{{#each test}}
76			<img src="{{abe type='image' key='test.img' desc='test_img' tab='default'}}" alt="">
77			{{/each}}
78
79			* BECOME:
80			{{#each test}}
81			<img data-abe-attr-test[index].img="src" data-abe-test[index].img="test[index].img" src="{{abe type='image' key='test.img' desc='test_img' tab='default' has-abe=1}}" alt="">
82			{{/each}}
83
84			* @param {[type]} template [description]
85			*/
86			export function addAbeDataAttrForHtmlAttributes(template, key) {
			0 ignored issues – show Unused Code introduced 2016-12-13 17:38 UTC by Report Bug Copy Issue Report The parameter `key` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
87			var text = template.replace(/<([A-Za-z]+)/g, '\nABE_SPLIT<$1')
88			let abeTagIntoAttribute = text.match(cmsData.regex.abeAsAttributePattern)
89
90			Array.prototype.forEach.call(abeTagIntoAttribute, (abeIntoTag) => {
91			let matchAbeTag = /({{abe.?[\s\S].?}})/g.exec(abeIntoTag)
92
93			if(matchAbeTag != null && matchAbeTag[1] != null) {
94			var toReplace = cmsTemplates.prepare.getAbeAttributeData(matchAbeTag[1], text, (abeIntoTag.split('=')[0]).trim(), abeIntoTag)
95
96			toReplace = toReplace.replace(
97			cmsData.regex.escapeTextToRegex(matchAbeTag[1]),
98			cmsTemplates.prepare.addHasAbeAttr(matchAbeTag[1])
99			)
100
101			text = text.replace(
102			cmsData.regex.escapeTextToRegex(abeIntoTag),
103			toReplace
104			)
105			}
106			})
107			text = text.replace(/\nABE_SPLIT</g, '<')
108
109			return text
110			}
111
112			/**
113			* Example:
114			*
115			*
116			* THIS:
117			{{abe type='data' key='data_key' source='select title from article' display='title' editable='true' tab='default'}}
118
119			{{#each data_key}}
120			{{title}}
121			{{/each}}
122
123			*
124			* BECOME THIS
125
126			{{abe type='data' key='data_key' source='select title from article' display='title' editable='true' tab='default'}}
127
128			{{#each data_key}}
129			{{title}}
130			{{/each}}<!-- [[data_key]] %7B%7B%23each%20data_key%7D%7D%0A%09%7B%7Btitle%7D%7D%0A%7B%7B/each%7D%7D -->
131
132			* @param {[type]} template [description]
133			* @param {[type]} json [description]
134			*/
135			export function addAbeSourceComment(template, json) {
136
137			// Don't know what it does...
138			if(typeof json.abe_source !== 'undefined' && json.abe_source !== null) {
139			var keys = Object.keys(json.abe_source)
140
141			for(var i in keys) {
			0 ignored issues – show Complexity introduced 2016-09-27 10:32 UTC by Report Bug Copy Issue Report A for in loop automatically includes the property of any prototype object, consider checking the key using `hasOwnProperty`. When iterating over the keys of an object, this includes not only the keys of the object, but also keys contained in the prototype of that object. It is generally a best practice to check for these keys specifically: var someObject; for (var key in someObject) { if ( ! someObject.hasOwnProperty(key)) { continue; // Skip keys from the prototype. } doSomethingWith(key); } Loading history...
142			var replaceEach = new RegExp(`<!-- \\[\\[${keys[i]}\\]\\][\\s\\S]*?-->`, 'g')
143			template = template.replace(replaceEach, '')
144
145			var patAttrSource = new RegExp(' ([A-Za-z0-9\-\_]+)=["\|\'].?({{' + keys[i] + '}}).?["\|\']', 'g')
146			var patAttrSourceMatch = template.match(patAttrSource)
147
148			if(patAttrSourceMatch != null) {
149			let checkEscapedRegex = /["\|'](.*?)["\|']/
150			let patAttrSourceInside = new RegExp('(\\S+)=["\']?((?:.(?!["\']?\\s+(?:\\S+)=\|[>"\']))+.)["\']?({{' + keys[i] + '}}).*?["\|\']', 'g')
151			Array.prototype.forEach.call(patAttrSourceMatch, (pat) => {
152			let patAttrSourceCheck = patAttrSourceInside.exec(pat)
153			if(patAttrSourceCheck != null) {
154
155			let checkEscaped = checkEscapedRegex.exec(patAttrSourceCheck[0])
156			if(checkEscaped != null && checkEscaped.length > 0) {
157			checkEscaped = escape(checkEscaped[1])
158			template = template.replace(
			0 ignored issues – show Bug introduced 2016-12-06 14:17 UTC by Report Bug Copy Issue Report The variable `template` is changed as part of the for-each loop for example by `template.replace(replaceEach, "")` on line `143`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history...
159			patAttrSourceCheck[0],
160			` data-abe-attr="${patAttrSourceCheck[1]}" data-abe-attr-escaped="${checkEscaped}" data-abe="${keys[i]}" ${patAttrSourceCheck[0]}`
			0 ignored issues – show introduced 2016-09-27 10:32 UTC by Report Bug Copy Issue Report The variable `i` is changed by the for-each loop on line 141. Only the value of the last iteration will be visible in this function if it is called outside of the loop. Loading history...
161			)
162			}
163			}
164			})
165			}
166
167			var eachSource = new RegExp(`({{#each ${keys[i]}}[\\s\\S a-z]*?{{\/each}})`, 'g')
168			var matches = template.match(eachSource)
169			if(typeof matches !== 'undefined' && matches !== null) {
170			Array.prototype.forEach.call(matches, (match) => {
171			template = template.replace(match, `${match}<!-- [[${keys[i]}]] ${cmsTemplates.encodeAbeTagAsComment(match)} -->`)
			0 ignored issues – show Bug introduced 2016-12-06 14:17 UTC by Report Bug Copy Issue Report The variable `template` is changed as part of the for-each loop for example by `template.replace(replaceEach, "")` on line `143`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history... introduced 2016-09-27 10:32 UTC by Report Bug Copy Issue Report The variable `i` is changed by the for-each loop on line 141. Only the value of the last iteration will be visible in this function if it is called outside of the loop. Loading history...
172			})
173			}
174			}
175			}
176
177			return template
178			}
179
180			/**
181			* THIS:
182			<span>{{abe type='text' key='text_visible'}}</span>
183
184			* BECOME:
185			<span><abe>{{abe type='text' key='text_visible'}}</abe></span>
186
187			* @param {[type]} template [description]
188			*/
189			export function addAbeHtmlTagBetweenAbeTags(template) {
190			var match
191			while (match = cmsData.regex.abePattern.exec(template)) {
192			template = template.replace(cmsData.regex.escapeTextToRegex(match[1], 'g'), '<abe>' + match[1].trim() + '</abe>')
193			}
194
195			return template
196			}
197
198			/**
199			* THIS:
200			[index].
201
202			* BECOME:
203			{{@index}}-
204
205			* @param {[type]} template [description]
206			* @return {[type]} [description]
207			*/
208			export function replaceAbeEachIndex(template) {
209			return template.replace(/\[index\]\./g, '{{@index}}-')
210			}
211
212			export function removeHiddenAbeTag(template) {
213			return template.replace(/(\{\{abe.visible=[\'\|\"]false.\}\})/g, '')
214			}
215
216			/**
217			* Remove {{abe type=*}} from html if attribute visible="false"
218			* @param {[type]} template [description]
219			* @return {[type]} [description]
220			*/
221			export function removeHandlebarsRawFromHtml(template) {
222			return template.replace(/\{\{\{\{\/?raw\}\}\}\}/g, '')
223			}
224
225			/**
226			* split {{#each}}...{{/each}} into an array
227			*
228			* @param {[type]} template [description]
229			* @return {[type]} [description]
230			*/
231			export function splitEachBlocks(template) {
232			var block
233			var blocks = []
234
235			while (block = cmsData.regex.blockPattern.exec(template)) {
236			blocks.push(block[1])
237			}
238
239			return blocks
240			}
241
242			export function indexEachBlocks(template, onlyHtml) {
243			// create an array of {{each}} blocks
244			var blocks = cmsTemplates.prepare.splitEachBlocks(template)
245
246			Array.prototype.forEach.call(blocks, (block) => {
247			var key = block.match(/#each (.*)\}\}/)[1]
248			var match
249
250			if(!onlyHtml) {
251
252			var voidData = {}
253			voidData[key] = [{}]
254			var blockCompiled = Handlebars.compile(block.replace(/{{abe (.*?)}}/g, '[[abe $1]]').replace(new RegExp(`\\.\\.\/${config.meta.name}`, 'g'), config.meta.name))
255			var blockHtml = blockCompiled(voidData, {data: {intl: config.intlData}}).replace(/\[\[abe (.*?)\]\]/g, '{{abe $1}}')
256
257			// je rajoute un data-abe-block avec index sur tous les tags html du bloc each
258			var textEachWithIndex = block.replace(/(<(?![\/])[A-Za-z0-9!-]*)/g, '$1 data-abe-block="' + key + '{{@index}}"')
259
260			// je remplace le block dans le texte par ça
261			template = template.replace(block, textEachWithIndex + `<!-- [[${key}]] ${cmsTemplates.encodeAbeTagAsComment(blockHtml)} -->`)
262			}
263
264			// Pour chaque tag Abe
265			while (match = cmsData.regex.abeTag.exec(block)) {
266			// template = cmsTemplates.prepare.insertAbeEach(template, match, key, cmsData.regex.eachBlockPattern.lastIndex - block.length, onlyHtml)
267			template = cmsTemplates.prepare.addAbeDictionnary(template, match[0], key)
			0 ignored issues – show Bug introduced 2016-12-13 17:38 UTC by Report Bug Copy Issue Report The variable `template` is changed as part of the while loop for example by `cmsTemplates.prepare.add...template, match.0, key)` on line `267`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history...
268			}
269			})
270
271			return template
272			}
273
274			/**
275			* split {{#each}}...{{/each}} into an array
276			*
277			* THIS:
278			{{abe type='text' key='test.title' desc='test title' tab='default'}}
279
280			* BECOME THIS:
281			{{abe dictionnary='test' type='text' key='test.title' desc='test title' tab='default'}}
282
283			*
284			* @param {[type]} template [description]
285			* @return {[type]} [description]
286			*/
287			export function addAbeDictionnary(template, match, key) {
288			if(cmsData.regex.isEachStatement(match)) return
			0 ignored issues – show Coding Style Best Practice introduced 2016-09-27 10:32 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
289
290			if(cmsData.regex.isBlockAbe(match)){
291			var abeDictionnary = match.replace(new RegExp('(key=[\'\|"])' + key + '.', 'g'), '$1' + key + '[index].')
292			.replace(/\{\{abe/, '{{abe dictionnary=\'' + key + '\'')
293
294			template = template.replace(match, abeDictionnary)
295			}
296
297			return template
298			}

AdFabConnect / abejs

Push — master ( c2f34b...8075f4 )

prepare.js ➔ ... ➔ ??? A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like