Ticket #3314 (closed feature: fixed)

Opened 2 years ago

Last modified 22 months ago

XYZ.js trips-up JavaScript minify tools

Reported by: mgleahy Owned by:
Priority: trivial Milestone: 2.11 Release
Component: Layer.XYZ Version: 2.11 RC1
Keywords: Cc:
State: Pullup

Description

Lines 144 and 146 (as-of 2.11-rc1) in lib/OpenLayers/Layer/XYZ.js both start with '/' as the first non-whitespace character. This trips up PHP JSMin, and possibly other tools that minify/compress JavaScript.

If the slashes were moved to the end of the previous lines, or if both lines 144 and 146 were merged with their preceeding lines, this would solve the problem with compressing this file with 3rd-party compression tools.

I'm not sure if this could be included in 2.11 before the final release, but it's a trivial change.

Change History

Changed 2 years ago by erilem

  • state set to Pullup

Changed 2 years ago by erilem

Changed 2 years ago by pgiraud

  • status changed from new to closed
  • resolution set to fixed

Changed 2 years ago by pgiraud

  • status changed from closed to reopened
  • resolution fixed deleted

Actually, the ticket should be left open.

Changed 22 months ago by ahocevar

  • status changed from reopened to closed
  • resolution set to fixed

in 2.11-rc2

Note: See TracTickets for help on using tickets.