PHP Warnings on Image Style definitions and library functions [#1338324]

Hi all

Great module :-) finally :-)

But :-) ...

Some messages

Notice: Undefined index: theme in flexslider_add() (line 288 of /sites/all/modules/flexslider/flexslider-7.x-1.x-dev-b549e1b/flexslider.module).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: theme in flexslider_add() (line 288 of /sites/all/modules/flexslider/flexslider-7.x-1.x-dev-b549e1b/flexslider.module).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: theme in flexslider_add() (line 288 of /sites/all/modules/flexslider/flexslider-7.x-1.x-dev-b549e1b/flexslider.module).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Undefined index: height in image_scale_dimensions() (line 157 of /modules/image/image.effects.inc).
Notice: Trying to get property of non-object in user_access() (line 790 of /modules/user/user.module).
Notice: Trying to get property of non-object in user_access() (line 802 of /modules/user/user.module).
Notice: Trying to get property of non-object in user_access() (line 803 of /modules/user/user.module).
Notice: Trying to get property of non-object in user_access() (line 809 of /modules/user/user.module).
Notice: Trying to get property of non-object in user_access() (line 812 of /modules/user/user.module).

I will disable messages, the error ones, for anonymous

If i can help say me how :-) (good instructions because usually only i translate but with good teacher i make miracles :-)

Best regards
ArchGalileu

Comment	File	Size	Author
#8	1338324_php-warnings-imagestyles-8.patch	3.35 KB	minorOffense
#8
#7	1338324_php-warnings-imagestyles-7.patch	3.05 KB	minorOffense
#7
#6	removed-user-acess-errors-#1338324-6.patch	2.74 KB	webankit
#6
#2	1338324_php-warnings-imagestyles.patch	757 bytes	minorOffense
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

minorOffense CreditAttribution: minorOffense commented 11 November 2011 at 16:01

Assigned:

Unassigned

» minorOffense

I'll have a look. Probably due to me messing with the templates a little...

Comment #2

minorOffense CreditAttribution: minorOffense commented 11 November 2011 at 16:08

Title:	Error messages	» PHP Warnings on Image Style definitions and library functions
Status:	Active	» Needs review

File	Size
1338324_php-warnings-imagestyles.patch	757 bytes

I renamed the issue to be more specific.

Can you try this patch and let me know? My dev environment is a little broken at the moment but I think this should fix things.

Comment #4

minorOffense CreditAttribution: minorOffense commented 12 November 2011 at 18:51

Status:

Needs review

» Needs work

Hmm, those are going to require some more digging. Can you tell me which module/submodules you have enabled?

Comment #5

webankit CreditAttribution: webankit commented 12 November 2011 at 23:03

user_acess() error come by simplying eabling the FlexSlider Module

Comment #6

webankit CreditAttribution: webankit commented 13 November 2011 at 21:46

File	Size
removed-user-acess-errors-#1338324-6.patch	2.74 KB

There was an error in path also as includes directory doesn't exist. Something strange but when i removed administer site configuration permission from access arguments the errors are gone, may be in D7 we don't need to add this permission.

Comment #7

minorOffense CreditAttribution: minorOffense commented 14 November 2011 at 15:33

Status:

Needs work

» Needs review

File	Size
1338324_php-warnings-imagestyles-7.patch	3.05 KB

I seem to be getting a 404 on that uploaded patch.

But here's my own attempt at combining the two. I removed the extra permission check inside hook_menu (only required administer flexslider now) and the previous fix.