@planning-inspectorate/dynamic-forms 3.3.2 → 3.3.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@planning-inspectorate/dynamic-forms",
-  "version": "3.3.2",
+  "version": "3.3.3",
   "description": "This package is for building GDS forms using a configuration-based approach.",
   "main": "./src/index.js",
   "type": "module",

package/src/components/manage-list/question.js

@@ -1,6 +1,6 @@
 import { Question } from '#question';
 import { Uuid } from '#src/lib/uuid.js';
-import { nunjucksEnv } from '#src/components/utils/nunjucks.js';
+import nunjucks from 'nunjucks';
 
 /**
  * @typedef {Object} ManageListQuestionParameters
@@ -107,7 +107,10 @@ export default class ManageListQuestion extends Question {
 		if (answer && Array.isArray(answer)) {
 			if (this.#showAnswersInSummary) {
 				const answers = answer.map((a) => this.#formatItemAnswers(a));
-				formattedAnswer = nunjucksEnv().render('components/manage-list/answer-summary-list.njk', { answers });
+				// note: nunjucks.render uses the last configured environment
+				// so we assume here that it is the one used by the main application and
+				// is configured for dynamic-forms and govuk components
+				formattedAnswer = nunjucks.render('components/manage-list/answer-summary-list.njk', { answers });
 			} else if (answer.length > 0) {
 				formattedAnswer = `${answer.length} ${this.title}`;
 			}

package/src/components/manage-list/question.test.js

@@ -91,6 +91,8 @@ describe('components/manage-list/question', () => {
 	});
 
 	it('should support showing all answers in the format list answer for summary', (ctx) => {
+		// nunjucks.render uses the last environment configured, so make sure one is configured
+		configureNunjucksTestEnv();
 		const { q } = questionWithManageQuestions(ctx, { showAnswersInSummary: true });
 		const answerForSummary = q.formatAnswerForSummary('section-1', mockJourney(), [{}, {}, {}]);
 		assert.strictEqual(answerForSummary.length, 1);

package/src/components/unit-option-entry/question.js

@@ -112,6 +112,9 @@ export default class UnitOptionEntryQuestion extends Question {
 				conditionalField.value = payload ? payload[conditionalField.fieldName] : existingValue;
 
 				optionData.conditional = {
+					// note: nunjucks.render uses the last configured environment
+					// so we assume here that it is the one used by the main application and
+					// is configured for dynamic-forms and govuk components
 					html: nunjucks.render(`./components/conditional/unit.njk`, {
 						payload,
 						...conditionalField,

package/src/questions/options-question.js

@@ -100,6 +100,9 @@ export default class OptionsQuestion extends Question {
 				conditionalField.value = answers[conditionalField.fieldName] || '';
 
 				optionData.conditional = {
+					// note: nunjucks.render uses the last configured environment
+					// so we assume here that it is the one used by the main application and
+					// is configured for dynamic-forms and govuk components
 					html: nunjucks.render(`./components/conditional/${conditionalField.type}.njk`, {
 						payload,
 						...conditionalField,

package/src/components/utils/nunjucks.js

@@ -1,20 +0,0 @@
-import path from 'path';
-import nunjucks from 'nunjucks';
-
-/**
- * Returns a nunjucks environment configured with the `src` folder
- * @returns {import('nunjucks').Environment}
- */
-export function nunjucksEnv() {
-	const utilsDir = path.dirname(new URL(import.meta.url).pathname);
-	const srcDir = path.resolve(utilsDir, '..', '..');
-
-	return nunjucks.configure([srcDir], {
-		// output with dangerous characters are escaped automatically
-		autoescape: true,
-		// automatically remove trailing newlines from a block/tag
-		trimBlocks: true,
-		// automatically remove leading whitespace from a block/tag
-		lstripBlocks: true
-	});
-}