enketo-transformer - npm Package Compare versions

Comparing version 2.1.4 to 2.1.5

CHANGELOG.md

@@ -6,2 +6,8 @@ ## Change Log
 
+## [2.1.5] - 2022-05-05
+
+##### Changed
+
+-   Fixed regression when transforming forms with no media mapping, when the form has binary defaults or references jr: URLs in markdown. (#152)
+
 ## [2.1.4] - 2022-05-04
@@ -8,0 +14,0 @@

package.json

 {
     "name": "enketo-transformer",
-    "version": "2.1.4",
+    "version": "2.1.5",
     "description": "Library/app that transforms ODK-compliant XForms into a format that Enketo can consume",
@@ -5,0 +5,0 @@ "license": "Apache-2.0",

README.md

@@ -1,2 +0,2 @@
-![coverage-shield-badge-1](https://img.shields.io/badge/coverage-98.31%25-brightgreen.svg)
+![coverage-shield-badge-1](https://img.shields.io/badge/coverage-97.86%25-brightgreen.svg)
 [![npm version](https://badge.fury.io/js/enketo-transformer.svg)](http://badge.fury.io/js/enketo-transformer) [![Build Status](https://travis-ci.org/enketo/enketo-transformer.svg?branch=master)](https://travis-ci.org/enketo/enketo-transformer) [![Dependency Status](https://david-dm.org/enketo/enketo-transformer.svg)](https://david-dm.org/enketo/enketo-transformer)
@@ -3,0 +3,0 @@

src/transformer.js

@@ -83,12 +83,8 @@ /**
 
-    let mediaMap = null;
+    const mediaMap = Object.fromEntries(
+        Object.entries(survey.media || {}).map((entry) =>
+            entry.map(escapeURLPath)
+        )
+    );
 
-    if (survey.media) {
-        mediaMap = Object.fromEntries(
-            Object.entries(survey.media).map((entry) =>
-                entry.map(escapeURLPath)
-            )
-        );
-    }
-
     return _parseXml(survey.xform)
@@ -505,13 +501,11 @@ .then((doc) => {
             if (original !== rendered) {
-                if (mediaMap != null) {
-                    const fragment = libxmljs.parseHtmlFragment(
-                        `<div class="temporary-root">${rendered}</div>`
-                    );
+                const fragment = libxmljs.parseHtmlFragment(
+                    `<div class="temporary-root">${rendered}</div>`
+                );
 
-                    rendered = _replaceMediaSources(fragment, mediaMap)
-                        .root()
-                        .childNodes()
-                        .map((node) => node.toString(false))
-                        .join('');
-                }
+                rendered = _replaceMediaSources(fragment, mediaMap)
+                    .root()
+                    .childNodes()
+                    .map((node) => node.toString(false))
+                    .join('');
 
@@ -518,0 +512,0 @@ key = `$$$${index}`;

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Native code

Supply chain risk

Contains native code (e.g., compiled binaries or shared libraries). Including native code can obscure malicious behavior.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

1750704

increased by0.05%

Lines of code

4605

increased by0.26%

Worsened metrics

Number of downloads

66

decreased by-60.48%

Number of medium supply chain risk alerts

16

increased by6.67%

No dependency changes