php warning illegal string offset form_id

Hi @johannschnagl

Could you please try this pre-release version 2.2.3 and see if the issue still occurs? Ensure that you’ve created a backup of your site before you update it.
Download URL: https://downloads.www.ads-software.com/plugin/ultimate-member.2.2.3-beta.zip

Regards,

Hi @champsupertramp
Thanks for the beta version. I just installed it at set an alarm in my elasticsearch that collects all the php logs. I’ll post it if the warnings show up again. As it is/was a sporadic failure it might take a couple of days and as I go for a short holiday it might even take some more time.

Regards

Looking forward to your reply.

Regards,

Hi @champsupertramp
I have not seen my error yet but discovered another one
When I install the beta version I won’t be able to see menu items of the type individual link. I can only see them if I have admin rights. I tried this in my test environment. The menu item were there. Then I installed the beta version and the menu items were gone.
Regards

Hi @johannschnagl

Please see the fixes for the Menu title as post title in the pinned topic:
https://www.ads-software.com/support/topic/ultimate-member-2-2-x-content-restriction-settings-and-post-titles/

Regards,

Hey there!

This thread has been inactive for a while so we’re going to go ahead and mark it Resolved. …Please feel free to re-open this thread by changing the Topic Status to ‘Not Resolved’ if any other questions come up and we’d be happy to help. ??

Regards,

Hi @champsupertramp

I just checken my logs again and I‘m sorry but the error showed up again even with the new beta version.

I‘m abroad but I‘ll be back on the weekend to do Dome more Tests.

Kind regards

Hans

Hi @johannschnagl

Please check the Release Candidate version at the link:
https://downloads.www.ads-software.com/plugin/ultimate-member.2.2.3-rc.1.zip

Let us know if you’re still encountering issues after the update.

Regards,

Hi @champsupertramp,

I just installed 2.2.4. Usually I get two to four errors per week. So I’ll check again in a couple of days

Regards

@champsupertramp
This morning the php warning occured again. I have the impression that the warnings occure not so often any more. But they are not gone yet.

Regards

• This reply was modified 3 years, 1 month ago by johannschnagl.
• This reply was modified 3 years, 1 month ago by johannschnagl.

@johannschnagl

This PHP error message may come from “Format submitted data for Info preview & Email template”:

Illegal string offset ‘form_id’ in xxx/wp-content/plugins/ultimate-member/includes/um-short-functions.php on line 705

@champsupertramp

You will get this PHP error message when “you’re trying to use a string as a full array”.

https://stackoverflow.com/questions/9869150/illegal-string-offset-warning-php

Line 699 in um-short-functions.php

$fields = UM()->query()->get_attr( 'custom_fields', $submitted_data['form_id'] );

may return an empty string which you are not testing for.

The UM function get_attr is calling the WP get_post_meta function where return variable is defined as:

* @return mixed An array of values if "$single" is false.
 *               The value of the meta field if "$single" is true.
 *               False for an invalid "$post_id" (non-numeric, zero, or negative value).
 *               An empty string if a valid but non-existing post ID is passed.

UM is testing for if ( isset( $fields ) ) { and then accessing the empty string as an array. Would be much better to also test for an array.

• This reply was modified 3 years, 1 month ago by missveronica.

@johannschnagl

I have now reported this bug for action by the UM developers:

https://github.com/ultimatemember/ultimatemember/issues/913

• This reply was modified 3 years, 1 month ago by missveronica.

@missveronicatv
In my registration form some fields are required. So it might not a manual registration but some bot trying to register.

Hi @missveronicatv

Thanks for reporting the issue. We’ll look into it.

Regards,