width or width makes no sense, width or height does [#506368]

A micro patch attached that changes template_preprocess_filefield_insert_widget(), because the following makes no sense:

        if (isset($action['data']['width']) || isset($action['data']['width'])) {

and should clearly be:

        if (isset($action['data']['width']) || isset($action['data']['height'])) {

Presumably this could lead to issues with presets where only a height is defined, if not fixed.

Comment	File	Size	Author
	width-width_to_width-height_micro.patch	915 bytes	adrinux

Comments

he/him

CreditAttribution: quicksketch commented 30 June 2009 at 22:13

Status:

Needs review

» Fixed

Thanks, committed!

14 July 2009 at 22:20

Status:

Fixed

» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.