Вы, вероятно, ошибаетесь:
Когда $width > $height
, это означает, что это пейзаж.Установка максимальной ширины до 800 означает (высота / ширина) * 800 = новая высота.С другой стороны, $height > $width
означает установку максимальной высоты на 800 и, таким образом, наличие (ширина / высота) * 800 новой ширины.
Сейчас вы используете как отношение высоты / ширины, а не наоборот.Пример:
Image: 1600 (w) x 1200 (h)
Type: Landscape
New Width: 800
New Height: (1200 (h) / 1600(w) * 800 (nw) = 600
Image 1200 (w) x 1600 (h)
Type: Portrait
New Height: 800
New Width: (1200 (w) / 1600(h) * 800 (nh) = 600
Надеюсь, вы понимаете, о чем я говорю, вы просто их поменяли :) Также обратите внимание, что вы умножаете на $ newheight вместо $ newheight1 для эскиза портрета