Fix ogc/filter

This only compiles if there is goog.provide('ol.Extent') in the typedefs file, and goog.require('ol.Extent') in this file. An alternative is to add an assert statement, but this means an extra dependency.

With this change there is no longer a need for the change to tasks/build.js
This commit is contained in:
Peter Robins
2016-05-07 19:03:02 +00:00
parent 535e8f5e35
commit ed34caa25a
3 changed files with 2 additions and 3 deletions

View File

@@ -18,6 +18,7 @@ goog.provide('ol.format.ogc.filter.IsNull');
goog.provide('ol.format.ogc.filter.IsBetween');
goog.provide('ol.format.ogc.filter.IsLike');
goog.require('ol.Extent');
goog.require('ol.Object');

View File

@@ -2,6 +2,7 @@
* File for all top-level (in the `ol` namespace) typedefs used by the compiler,
* and referenced by JSDoc.
*/
goog.provide('ol.Extent');
/**

View File

@@ -207,9 +207,6 @@ function build(config, paths, callback) {
} else {
log.info('ol', 'Compiling ' + paths.length + ' sources');
options.compile.js = paths.concat(options.compile.js || []);
// typedefs file has no `goog.provide`s, so is ignored by closure-util
// when calculating dependencies. So it's added here as a compiler option.
options.compile.js.push('src/ol/typedefs.js');
closure.compile(options, callback);
}
}