Simplify orientation check.

blueimp · blueimp · commit 4a6fe058f17f · 2020-04-08T18:32:27.000+09:00
diff --git a/js/load-image-orientation.js b/js/load-image-orientation.js
@@ -59,7 +59,7 @@
     var styleWidth = canvas.style.width
     var styleHeight = canvas.style.height
     var orientation = options.orientation
-    if (!orientation || orientation > 8) {
+    if (!(orientation > 1 && orientation < 9)) {
       return
     }
     if (orientation > 4) {
@@ -118,7 +118,7 @@
     if (orientation === true && data && data.exif) {
       orientation = data.exif.get('Orientation')
     }
-    if (!orientation || orientation > 8 || orientation === 1) {
+    if (!(orientation > 1 && orientation < 9)) {
       return options
     }
     newOptions = {}

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@`
`59`	`59`	`var styleWidth = canvas.style.width`
`60`	`60`	`var styleHeight = canvas.style.height`
`61`	`61`	`var orientation = options.orientation`
`62`		`- if (!orientation \|\| orientation > 8) {`
	`62`	`+ if (!(orientation > 1 && orientation < 9)) {`
`63`	`63`	`return`
`64`	`64`	`}`
`65`	`65`	`if (orientation > 4) {`
`@@ -118,7 +118,7 @@`
`118`	`118`	`if (orientation === true && data && data.exif) {`
`119`	`119`	`orientation = data.exif.get('Orientation')`
`120`	`120`	`}`
`121`		`- if (!orientation \|\| orientation > 8 \|\| orientation === 1) {`
	`121`	`+ if (!(orientation > 1 && orientation < 9)) {`
`122`	`122`	`return options`
`123`	`123`	`}`
`124`	`124`	`newOptions = {}`