[#4289] image field error

OK. I have fixed the issue in framework file as we are in production. I have made some testing and it is doing removal of partial created user from system using through CB functionality in case partial user got created during registration. I have used the existing code in cb.tables.php(just changed the order of execution of code) but since I am not an expert in CB framework code, I am not sure that it is proper fix considering the dependency that CB performs operation before and after the point where I added fix (calling post trigger etc).

Will it possible for you to share review if that is correct fix. File is attached. It is a request and not a push.

There is one fatal exception coming in cbactivity delete user, when below code is getting fired through CB delete user functionality.

_PLUGINS->trigger( 'onAfterDeleteUser', array( $user, true ) );

"Call to undefined method stdClass::get() in components\com_comprofiler\plugin\user\plug_cbactivity\plugin.cbactivity.php on line 576"

On line 576 in cbactivity.php, I debugged user and it does have a id field with value.

Thanks,
Amit

file attached. please rename ph_ to php

I am not able to attach php file or zip file as www.joomlapolis.com/forum page breaks up and display blank page.

Thanks,
Amit

Will it possible for you to share review if that is correct fix. File is attached. It is a request and not a push.

That is not the correct fix. You shouldn't be deleting anything during registration. The correct fix is a fix to the image field validation function to validate the extension and mime type, which prevents registration from happening to begin with. If you simply want to delete a user on error on that trigger you can do so using CB Auto Actions without having to touch the core code.

There is one fatal exception coming in cbactivity delete user, when below code is getting fired through CB delete user functionality.

That's already fixed in latest CB Activity release.

I am not able to attach php file or zip file as www.joomlapolis.com/forum page breaks up and display blank page.

Our site security does not allow certain files to prevent malicious intent as PHP files are executable so they can not be uploaded.

Hi kyle,

Thanks for your inputs. I will update the Cb activity plugin. CB Auto Actions post the partial user creation are not being fired. Thats why we found this issue as on after user registration trigger is not called. Also I could not see any trigger for on error while user registration too. I checked this cb auto action on error first thing before looking for code change.

Thanks,
Amit

Sorry, I don't have anything more to suggest. The issue is fixed for next release. Until then you're welcome to just ignore it as it's not very critical due to the chances of someone uploading the wrong extension is pretty slim (it handles most common formats fine: jpg, jpeg, gif, and png).