diff --git a/modules/cms/widgets/MediaManager.php b/modules/cms/widgets/MediaManager.php index 0592a716c..ae5b23922 100644 --- a/modules/cms/widgets/MediaManager.php +++ b/modules/cms/widgets/MediaManager.php @@ -1179,9 +1179,11 @@ class MediaManager extends WidgetBase throw new SystemException('Invalid selection data.'); } - if (!ctype_digit($selectionData[$paramName])) { + if (!is_numeric($selectionData[$paramName])) { throw new SystemException('Invalid selection data.'); } + + $selectionData[$paramName] = (int) $selectionData[$paramName]; } $sessionDirectoryPath = $this->getCropSessionDirPath($cropSessionKey); diff --git a/modules/cms/widgets/mediamanager/assets/js/mediamanager.imagecroppopup.js b/modules/cms/widgets/mediamanager/assets/js/mediamanager.imagecroppopup.js index 369961f1e..a63130775 100644 --- a/modules/cms/widgets/mediamanager/assets/js/mediamanager.imagecroppopup.js +++ b/modules/cms/widgets/mediamanager/assets/js/mediamanager.imagecroppopup.js @@ -202,7 +202,7 @@ mode = this.getSelectionMode() switch (mode) { - case 'fixed-ratio' : + case 'fixed-ratio' : this.jCrop.setOptions({ aspectRatio: width/height, minSize: [0, 0], @@ -210,7 +210,7 @@ allowResize: true }) break - case 'fixed-size' : + case 'fixed-size' : this.jCrop.setOptions({ aspectRatio: 0, minSize: [width, height], @@ -218,7 +218,7 @@ allowResize: false }) break - case 'normal' : + case 'normal' : this.jCrop.setOptions({ aspectRatio: 0, minSize: [0, 0], @@ -314,7 +314,7 @@ var data = { cropSessionKey: this.$popupElement.find('input[name=cropSessionKey]').val(), path: this.$popupElement.find('input[name=path]').val() - } + } $.oc.stripeLoadIndicator.show() $(ev.target).request(this.options.alias+'::onResizeImage', { @@ -338,7 +338,7 @@ this.$popupElement.find('span[data-label=width]').text(width) this.$popupElement.find('span[data-label=height]').text(hegiht) - + this.$popupElement.find('input[data-control=dimension-width]').val(width) this.$popupElement.find('input[data-control=dimension-height]').val(hegiht) @@ -370,8 +370,8 @@ } this.selectionSizeLabel.setAttribute('class', '') - this.selectionSizeLabel.querySelector('[data-label=selection-width]').textContent = width - this.selectionSizeLabel.querySelector('[data-label=selection-height]').textContent = height + this.selectionSizeLabel.querySelector('[data-label=selection-width]').textContent = parseInt(width) + this.selectionSizeLabel.querySelector('[data-label=selection-height]').textContent = parseInt(height) } // EVENT HANDLERS