Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jsrender.min.js.map truncated #445

Closed
Jerry2048 opened this issue Apr 29, 2020 · 6 comments
Closed

jsrender.min.js.map truncated #445

Jerry2048 opened this issue Apr 29, 2020 · 6 comments

Comments

@Jerry2048
Copy link

Somehow jsrender.min.js.map was truncated between v1.0.5 and v1.0.6. The newer one ends abruptly in the middle of the comment "// Test is satisfied, so render content on current context", causing an improperly formed JSON file and a warning when debugging my website in Chrome with F12: "DevTools failed to load SourceMap: Could not parse content for http://[name of site]/js/jsrender.min.js.map: Unexpected end of JSON input".

@BorisMoore
Copy link
Owner

Thanks for calling this out. I hadn't realized there was a problem. I don't know how it got truncated. I look into it, but meantime you can get a valid jsrender.min.js.map file here:

https://www.jsviews.com/download/jsrender.min.js.map

(also linked from the https://www.jsviews.com/#download page).

Let me know if that version works correctly for you...

@Jerry2048
Copy link
Author

Jerry2048 commented Apr 30, 2020 via email

@Jerry2048
Copy link
Author

Jerry2048 commented May 4, 2020 via email

BorisMoore added a commit to BorisMoore/jsviews.com that referenced this issue Jul 12, 2020
Feature improvements for JsRender on Node.js:

  (See issue BorisMoore/jsrender#357)

- File based templates for JsRender on Node.js now allow
  absolute paths

  See the updated documentation topic:
  https://www.jsviews.com/#node/filetmpls@htmlfile

- The renderFile() method for JsRender on Node.js
  now allows passing of context and helpers as well
  as data

  See the new documentation topic:
  https://www.jsviews.com/#node/filetmpls@renderfilehlelpers

Performance optimization changes:

- BorisMoore/jsviews#448
  $.observe/$.unobserve performance degradation over time

- BorisMoore/jsviews#447
  removeView performance

Several bug fixes, including:

- BorisMoore/jsviews#446
  Observable change error, when loading jquery.observable.js before
  jsrender.js, and using contextual parameters and sorted {{for}}

- BorisMoore/jsviews#440
  linked tag property changed handler called when it shouldn't

- BorisMoore/jsviews#445
  jsrender.min.js.map truncated

Additional small corrections or improvements to documentation
and some additional unit tests...
BorisMoore added a commit to BorisMoore/jsrender that referenced this issue Jul 12, 2020
Feature improvements for JsRender on Node.js:

  (See issue #357)

- File based templates for JsRender on Node.js now allow
  absolute paths

  See the updated documentation topic:
  https://www.jsviews.com/#node/filetmpls@htmlfile

- The renderFile() method for JsRender on Node.js
  now allows passing of context and helpers as well
  as data

  See the new documentation topic:
  https://www.jsviews.com/#node/filetmpls@renderfilehlelpers

Bug fix:

- BorisMoore/jsviews#445
  jsrender.min.js.map truncated

Additional small corrections or improvements to documentation
and some additional unit tests...
BorisMoore added a commit that referenced this issue Jul 12, 2020
Feature improvements for JsRender on Node.js:

  (See issue BorisMoore/jsrender#357)

- File based templates for JsRender on Node.js now allow
  absolute paths

  See the updated documentation topic:
  https://www.jsviews.com/#node/filetmpls@htmlfile

- The renderFile() method for JsRender on Node.js
  now allows passing of context and helpers as well
  as data

  See the new documentation topic:
  https://www.jsviews.com/#node/filetmpls@renderfilehlelpers

Performance optimization changes:

- #448
  $.observe/$.unobserve performance degradation over time

- #447
  removeView performance

Several bug fixes, including:

- #446
  Observable change error, when loading jquery.observable.js before
  jsrender.js, and using contextual parameters and sorted {{for}}

- #440
  linked tag property changed handler called when it shouldn't

- #445
  jsrender.min.js.map truncated

Additional small corrections or improvements to documentation
and some additional unit tests...
@BorisMoore
Copy link
Owner

This has been fixed in new release v1.0.7

@BorisMoore
Copy link
Owner

This truncated .map file has unfortunately happened again on v1.0.8. I'll probably need to release a new version (v1.0.9) to fix this on the CDNJS site.

But meantime a valid jsrender.min.js.map file for v1.0.8 is available here:

https://www.jsviews.com/download/jsrender.min.js.map

(also linked from the https://www.jsviews.com/#download page).

@BorisMoore BorisMoore reopened this Nov 27, 2020
BorisMoore added a commit to BorisMoore/jsviews.com that referenced this issue Dec 5, 2020
Replacement of truncated version of jsrender.min.js.map with correct
non-truncated version.

This update is otherwise identical to v1.0.8.

Details:

  In v1.0.8 there was an erroneously truncated version of jsrender.min.js.map at the
  following locations:
  - the JsRender and JsViews GitHub repositories
  - the CDNs:
    https://cdnjs.cloudflare.com/ajax/libs/jsviews/1.0.8/jsrender.min.js.map
    https://cdnjs.cloudflare.com/ajax/libs/jsrender/1.0.8/jsrender.min.js.map

  (Note that the v1.0.8 version at https://www.jsviews.com/download/jsrender.min.js.map
  was the correct complete file - also available here:
  https://raw.githubusercontent.com/BorisMoore/jsviews.com/febf6d8975dc66e18ff2b600f565c42ed286e4b4/download/jsrender.min.js.map)

With this update, all truncated versions are replaced by the correct complete
file.

See BorisMoore/jsviews#445 (comment)
and BorisMoore/jsrender#362
BorisMoore added a commit to BorisMoore/jsrender that referenced this issue Dec 5, 2020
Replacement of truncated version of jsrender.min.js.map with correct
non-truncated version.

This update is otherwise identical to v1.0.8.

Details:

  In v1.0.8 there was an erroneously truncated version of jsrender.min.js.map in the
  JsRender GitHub repositories, as well as on the CDN:
  - https://cdnjs.cloudflare.com/ajax/libs/jsrender/1.0.8/jsrender.min.js.map

  (Note that the v1.0.8 version at https://www.jsviews.com/download/jsrender.min.js.map
  was the correct complete file - also available here:
  https://raw.githubusercontent.com/BorisMoore/jsviews.com/febf6d8975dc66e18ff2b600f565c42ed286e4b4/download/jsrender.min.js.map)

With this update, all truncated versions are replaced by the correct complete
file.

See BorisMoore/jsviews#445 (comment)
and #362
BorisMoore added a commit that referenced this issue Dec 5, 2020
Replacement of truncated version of jsrender.min.js.map with correct
non-truncated version.

This update is otherwise identical to v1.0.8.

Details:

  In v1.0.8 there was an erroneously truncated version of jsrender.min.js.map in the
  JsViews GitHub repositories, as well as on the CDN:
  - https://cdnjs.cloudflare.com/ajax/libs/jsviews/1.0.8/jsrender.min.js.map

  (Note that the v1.0.8 version at https://www.jsviews.com/download/jsrender.min.js.map
  was the correct complete file - also available here:
  https://raw.githubusercontent.com/BorisMoore/jsviews.com/febf6d8975dc66e18ff2b600f565c42ed286e4b4/download/jsrender.min.js.map)

With this update, all truncated versions are replaced by the correct complete
file.

See #445 (comment)
and BorisMoore/jsrender#362
@BorisMoore
Copy link
Owner

This has been resolved in v1.0.9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants