Merge pull request #11865 from mike-000/patch-19
Use ol/array#includes instead of Array.prototype.includes()
This commit is contained in:
@@ -3,6 +3,7 @@
|
|||||||
*/
|
*/
|
||||||
|
|
||||||
import {assert} from '../asserts.js';
|
import {assert} from '../asserts.js';
|
||||||
|
import {includes} from '../array.js';
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @typedef {Object} PreferredOptions
|
* @typedef {Object} PreferredOptions
|
||||||
@@ -245,10 +246,10 @@ function generateVersion3Options(iiifInfo) {
|
|||||||
iiifInfo.imageInfo.preferredFormats.length > 0
|
iiifInfo.imageInfo.preferredFormats.length > 0
|
||||||
? iiifInfo.imageInfo.preferredFormats
|
? iiifInfo.imageInfo.preferredFormats
|
||||||
.filter(function (format) {
|
.filter(function (format) {
|
||||||
return ['jpg', 'png', 'gif'].includes(format);
|
return includes(['jpg', 'png', 'gif'], format);
|
||||||
})
|
})
|
||||||
.reduce(function (acc, format) {
|
.reduce(function (acc, format) {
|
||||||
return acc === undefined && formats.includes(format)
|
return acc === undefined && includes(formats, format)
|
||||||
? format
|
? format
|
||||||
: acc;
|
: acc;
|
||||||
}, undefined)
|
}, undefined)
|
||||||
@@ -456,16 +457,16 @@ class IIIFInfo {
|
|||||||
sizes: imageOptions.sizes,
|
sizes: imageOptions.sizes,
|
||||||
format:
|
format:
|
||||||
options.format !== undefined &&
|
options.format !== undefined &&
|
||||||
imageOptions.formats.includes(options.format)
|
includes(imageOptions.formats, options.format)
|
||||||
? options.format
|
? options.format
|
||||||
: imageOptions.preferredFormat !== undefined
|
: imageOptions.preferredFormat !== undefined
|
||||||
? imageOptions.preferredFormat
|
? imageOptions.preferredFormat
|
||||||
: 'jpg',
|
: 'jpg',
|
||||||
supports: imageOptions.supports,
|
supports: imageOptions.supports,
|
||||||
quality:
|
quality:
|
||||||
options.quality && imageOptions.qualities.includes(options.quality)
|
options.quality && includes(imageOptions.qualities, options.quality)
|
||||||
? options.quality
|
? options.quality
|
||||||
: imageOptions.qualities.includes('native')
|
: includes(imageOptions.qualities, 'native')
|
||||||
? 'native'
|
? 'native'
|
||||||
: 'default',
|
: 'default',
|
||||||
resolutions: Array.isArray(imageOptions.resolutions)
|
resolutions: Array.isArray(imageOptions.resolutions)
|
||||||
|
|||||||
+13
-12
@@ -9,6 +9,7 @@ import {DEFAULT_TILE_SIZE} from '../tilegrid/common.js';
|
|||||||
import {Versions} from '../format/IIIFInfo.js';
|
import {Versions} from '../format/IIIFInfo.js';
|
||||||
import {assert} from '../asserts.js';
|
import {assert} from '../asserts.js';
|
||||||
import {getTopLeft} from '../extent.js';
|
import {getTopLeft} from '../extent.js';
|
||||||
|
import {includes} from '../array.js';
|
||||||
import {toSize} from '../size.js';
|
import {toSize} from '../size.js';
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -111,11 +112,11 @@ class IIIF extends TileImage {
|
|||||||
const supportsArbitraryTiling =
|
const supportsArbitraryTiling =
|
||||||
supports != undefined &&
|
supports != undefined &&
|
||||||
Array.isArray(supports) &&
|
Array.isArray(supports) &&
|
||||||
(supports.includes('regionByPx') || supports.includes('regionByPct')) &&
|
(includes(supports, 'regionByPx') || includes(supports, 'regionByPct')) &&
|
||||||
(supports.includes('sizeByWh') ||
|
(includes(supports, 'sizeByWh') ||
|
||||||
supports.includes('sizeByH') ||
|
includes(supports, 'sizeByH') ||
|
||||||
supports.includes('sizeByW') ||
|
includes(supports, 'sizeByW') ||
|
||||||
supports.includes('sizeByPct'));
|
includes(supports, 'sizeByPct'));
|
||||||
|
|
||||||
let tileWidth, tileHeight, maxZoom;
|
let tileWidth, tileHeight, maxZoom;
|
||||||
|
|
||||||
@@ -270,10 +271,10 @@ class IIIF extends TileImage {
|
|||||||
regionParam = 'full';
|
regionParam = 'full';
|
||||||
} else if (
|
} else if (
|
||||||
!supportsArbitraryTiling ||
|
!supportsArbitraryTiling ||
|
||||||
supports.includes('regionByPx')
|
includes(supports, 'regionByPx')
|
||||||
) {
|
) {
|
||||||
regionParam = regionX + ',' + regionY + ',' + regionW + ',' + regionH;
|
regionParam = regionX + ',' + regionY + ',' + regionW + ',' + regionH;
|
||||||
} else if (supports.includes('regionByPct')) {
|
} else if (includes(supports, 'regionByPct')) {
|
||||||
const pctX = formatPercentage((regionX / width) * 100),
|
const pctX = formatPercentage((regionX / width) * 100),
|
||||||
pctY = formatPercentage((regionY / height) * 100),
|
pctY = formatPercentage((regionY / height) * 100),
|
||||||
pctW = formatPercentage((regionW / width) * 100),
|
pctW = formatPercentage((regionW / width) * 100),
|
||||||
@@ -282,16 +283,16 @@ class IIIF extends TileImage {
|
|||||||
}
|
}
|
||||||
if (
|
if (
|
||||||
version == Versions.VERSION3 &&
|
version == Versions.VERSION3 &&
|
||||||
(!supportsArbitraryTiling || supports.includes('sizeByWh'))
|
(!supportsArbitraryTiling || includes(supports, 'sizeByWh'))
|
||||||
) {
|
) {
|
||||||
sizeParam = sizeW + ',' + sizeH;
|
sizeParam = sizeW + ',' + sizeH;
|
||||||
} else if (!supportsArbitraryTiling || supports.includes('sizeByW')) {
|
} else if (!supportsArbitraryTiling || includes(supports, 'sizeByW')) {
|
||||||
sizeParam = sizeW + ',';
|
sizeParam = sizeW + ',';
|
||||||
} else if (supports.includes('sizeByH')) {
|
} else if (includes(supports, 'sizeByH')) {
|
||||||
sizeParam = ',' + sizeH;
|
sizeParam = ',' + sizeH;
|
||||||
} else if (supports.includes('sizeByWh')) {
|
} else if (includes(supports, 'sizeByWh')) {
|
||||||
sizeParam = sizeW + ',' + sizeH;
|
sizeParam = sizeW + ',' + sizeH;
|
||||||
} else if (supports.includes('sizeByPct')) {
|
} else if (includes(supports, 'sizeByPct')) {
|
||||||
sizeParam = 'pct:' + formatPercentage(100 / scale);
|
sizeParam = 'pct:' + formatPercentage(100 / scale);
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
|
|||||||
Reference in New Issue
Block a user