Ticket #3469 (closed feature: fixed)

Opened 21 months ago

Last modified 20 months ago

remove unneeded requires from OpenLayers/Protocol/HTTP.js

Reported by: fredj Owned by: erilem
Priority: minor Milestone: 2.12 Release
Component: Protocol.HTTP Version: SVN
Keywords: Cc:
State: Commit

Description

The class does not use any of these classes, the following requires can be removed:

Index: lib/OpenLayers/Protocol/HTTP.js
===================================================================
--- lib/OpenLayers/Protocol/HTTP.js	(revision 12260)
+++ lib/OpenLayers/Protocol/HTTP.js	(working copy)
@@ -4,12 +4,7 @@
  * full text of the license. */
 
 /**
- * @requires OpenLayers/Console.js
  * @requires OpenLayers/Protocol.js
- * @requires OpenLayers/Feature/Vector.js
- * @requires OpenLayers/Filter/Spatial.js
- * @requires OpenLayers/Filter/Comparison.js
- * @requires OpenLayers/Filter/Logical.js
  * @requires OpenLayers/Request/XMLHttpRequest.js
  */

Attachments

3469.0.patch Download (1.1 KB) - added by fredj 21 months ago.
move some @requires from Protocol/HTTP.js to Format/QueryStringFilter.js

Change History

Changed 21 months ago by fredj

  • state set to Review
  • version changed from 2.10 to SVN

Changed 21 months ago by erilem

I think Format/QueryStringFilter.js miss @requires directives (e.g. Spatial.js and Logical.js).

Changed 21 months ago by fredj

move some @requires from Protocol/HTTP.js to Format/QueryStringFilter.js

Changed 20 months ago by fredj

  • owner set to erilem

Changed 20 months ago by erilem

  • state changed from Review to Commit

Changed 20 months ago by fredj

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

(In [12383]) remove unneeded requires from OpenLayers/Protocol/HTTP.js r=erilem (closes #3469)

Note: See TracTickets for help on using tickets.