Commit d62b32f9 authored by Pedro Eduardo Trujillo's avatar Pedro Eduardo Trujillo
Browse files

Crea categoría de helpers y controla ausencias 

Se genera nuevo grupo de helpers dedicados a modificar el comportamiento
de los condicionales básicos en base a sub-expresiones.

Crea nuevo helper para comprobar que un valor sea 'falsy' pero también
definido. Es decir, excluye el valor 'undefined' de la lista de valores
considerados 'falsy'.

Se aplica el nuevo helper a la info y listado de documentos, que
mostraba incorrectamente que estaban deshabilitados en algunos listados
con campos reducidos en las respuestas.
parent ee91eb98

grunt/config/handlebars.js

+4 −1
Original line number Diff line number Diff line
@@ -79,7 +79,10 @@ module.exports = function(grunt) { 
						Image: true,



						// string

						Capitalize: true

						Capitalize: true,



						// subExpression

						PropertyIsDefinedAndFalse: true

					},

					knownHelpersOnly: true

				}

src/Document/Info.hbs

+2 −2
Original line number Diff line number Diff line
@@ -89,7 +89,7 @@ 
					</div>

				</div>

			{{/if}}

			{{#unless data.enabled}}

			{{#if (PropertyIsDefinedAndFalse data 'enabled')}}

				<div>

					<div class="rowLeft inlineRow {{RowColBibliography 'left'}}">

						<span title="{{i18n.enabled}}">{{i18n.enabled}}</span>
@@ -99,7 +99,7 @@ 
						<p>{{i18n.disabled}}</p>

					</div>

				</div>

			{{/unless}}

			{{/if}}

			{{#if data.internalUrl}}

				<div>

					<div class="rowLeft inlineRow {{RowColBibliography 'left'}}">

src/Document/List.hbs

+2 −2
Original line number Diff line number Diff line
@@ -19,9 +19,9 @@ 
		{{ChkIsNull data.title}}

	</span>

	<span class='badgesContainer'>

		{{#unless data.enabled}}

		{{#if (PropertyIsDefinedAndFalse data 'enabled')}}

			{{ItemEnabledStatus data i18n}}

		{{/unless}}

		{{/if}}

		{{#if data.internalUrl}}

			{{DocumentInternalUrlStatus data i18n}}

		{{/if}}

src/Helpers/subExpression.js

0 → 100644
+11 −0
Original line number Diff line number Diff line 
define([], function() {



	'use strict';



	return {

		PropertyIsDefinedAndFalse: function(data, propertyName) {



			return data && data[propertyName] !== undefined && !data[propertyName];

		}

	};

});

src/main.js

+6 −0
Original line number Diff line number Diff line
@@ -3,12 +3,14 @@ define([ 
	, 'handlebars/handlebars.runtime.min'

	, 'templates/Helpers/customParser'

	, 'templates/Helpers/string'

	, 'templates/Helpers/subExpression'

	, 'templates/Helpers/legacy'

], function(

	aspect

	, handlebars

	, customParserHelpers

	, stringHelpers

	, subExpressionHelpers

	, legacyHelpers

) {
@@ -20,6 +22,10 @@ define([ 
		handlebars.registerHelper(stringHelperName, stringHelpers[stringHelperName]);

	}



	for (var subExpressionHelperName in subExpressionHelpers) {

		handlebars.registerHelper(subExpressionHelperName, subExpressionHelpers[subExpressionHelperName]);

	}



	for (var legacyHelperName in legacyHelpers) {

		handlebars.registerHelper(legacyHelperName, legacyHelpers[legacyHelperName]);

	}