refactor(routes)!: rename removeAlt param to remove_alt

BREAKING CHANGE: `removeAlt` query string param renamed to `remove_alt`
Fdawgs · Nov 25, 2024 · f740eda · f740eda
1 parent b19dd94
commit f740eda
Show file tree

Hide file tree

Showing 8 changed files with 8 additions and 8 deletions.
diff --git a/src/routes/docx/html/index.js b/src/routes/docx/html/index.js
@@ -73,7 +73,7 @@ async function route(server, options) {
 			const html = await server.docxToHtml(req.body);
 			const tidiedHtml = await server.tidyHtml(html, {
 				language: req.query.language,
-				removeAlt: req.query.removeAlt,
+				removeAlt: req.query.remove_alt,
 			});
 
 			res.type("text/html; charset=utf-8");

diff --git a/src/routes/docx/html/route.test.js b/src/routes/docx/html/route.test.js
@@ -73,7 +73,7 @@ describe("DOCX-to-HTML route", () => {
 			url: "/",
 			body: await readFile(filePath),
 			query: {
-				removeAlt: "true",
+				remove_alt: "true",
 			},
 			headers: {
 				accept: "application/json, text/html",

diff --git a/src/routes/docx/html/schema.js b/src/routes/docx/html/schema.js
@@ -57,7 +57,7 @@ const docxToHtmlPostSchema = {
 				.default("en")
 		)
 		.prop(
-			"removeAlt",
+			"remove_alt",
 			S.boolean().description(
 				"Set the `alt` attribute in `<img>` tags to an empty string"
 			)

diff --git a/src/routes/pdf/html/index.js b/src/routes/pdf/html/index.js
@@ -76,7 +76,7 @@ async function route(server, options) {
 			);
 			const tidiedHtml = await server.tidyHtml(embeddedHtml, {
 				language: req.query.language,
-				removeAlt: req.query.removeAlt,
+				removeAlt: req.query.remove_alt,
 			});
 
 			return server.tidyCss(tidiedHtml, {

diff --git a/src/routes/pdf/html/route.test.js b/src/routes/pdf/html/route.test.js
@@ -21,7 +21,7 @@ const queryStrings = generateCombos([
 	{ fonts: "Arial" },
 	{ ignoreImages: "true" },
 	{ noDrm: "true" },
-	{ removeAlt: "true" },
+	{ remove_alt: "true" },
 ]);
 
 describe("PDF-to-HTML route", () => {

diff --git a/src/routes/pdf/html/schema.js b/src/routes/pdf/html/schema.js
@@ -103,7 +103,7 @@ const pdfToHtmlPostSchema = {
 				.maxLength(127)
 		)
 		.prop(
-			"removeAlt",
+			"remove_alt",
 			S.boolean().description(
 				"Set the `alt` attribute in `<img>` tags to an empty string"
 			)

diff --git a/src/routes/rtf/html/index.js b/src/routes/rtf/html/index.js
@@ -75,7 +75,7 @@ async function route(server, options) {
 				req.conversionResults.body,
 				{
 					language: req.query.language,
-					removeAlt: req.query.removeAlt,
+					removeAlt: req.query.remove_alt,
 				}
 			);
 

diff --git a/src/routes/rtf/html/schema.js b/src/routes/rtf/html/schema.js
@@ -52,7 +52,7 @@ const rtfToHtmlPostSchema = {
 				.default("en")
 		)
 		.prop(
-			"removeAlt",
+			"remove_alt",
 			S.boolean().description(
 				"Set the `alt` attribute in `<img>` tags to an empty string"
 			)