Separate Configuration code into a different function

[Israel-Laguan]

Problem

This kind of code is adequate for separate into another function:

templatestringparser/index.js

Lines 19 to 24 in 29fec42

	const {openingbracket, closingbracket, trim} = config
	// setting default settings where config key is missing
	var obk = (openingbracket) ? openingbracket : settings().openingbracket
	var cbk = (closingbracket) ? closingbracket : settings().closingbracket
	var trm = (trim !== undefined && trim) ? trim : settings().trim

If the current way to manage configuration logic if the options grow in number the code would be larger and you are on risk of breaking the Single Responsibility Principle. Also it makes less obvious the purpose of the function.

Proposal

Make configuration lines move to another function. The logic can be in settings function but I suggest a new function instead.

[tunjioye]

WOW, This is a brilliant observation. We will definitely implement this Proposal soon.