codepuncher

Hi there,

We are also experiencing the same issue in v2.1.5.

Downgrading to 2.1.4 fixes the issue.

I presume the refactor in 2.1.5 has broke it.

This happens on multiple websites.

We are not overriding the tabs PHP template.

The issue exists here: https://github.com/woocommerce/woocommerce/blob/41cf2b285e8ee690b1ea804b6a774345d7144225/plugins/woocommerce/templates/single-product/tabs/tabs.php#L36

When do you plan to resolve the deprecation warnings?

I am getting this error on many sites for a long time. Please can this be fixed?

I too am seeing this issue on Multisite network activation.

Is there a fix for this yet?

I have opened an issue on the GitHub repo https://github.com/EvanHerman/yikes-inc-easy-mailchimp-extender/issues/906 and also submitted a PR to resolve this here https://github.com/EvanHerman/yikes-inc-easy-mailchimp-extender/pull/907

Hope plugin author will merge and publish new release soon.

Hi @webholism,

Yes, the latest version resolves this issue thanks.

However, we are presented with 2 final issues to resolve:

Issue 1:
Notice: Trying to access array offset on value of type null in gf-form-multicolumn/includes/public/WH_GF_Multicolumn_Public_Form_Current.php on line 152

Caused by:

$divisor = ( $this->rowColumnArray[$this->rowCounter] > 0 ) ?
    $this->rowColumnArray[$this->rowCounter] : 1;

Why:
This assumes that $this->rowColumnArray is an array. However, it is defined without a value which results in it being null: private $rowColumnArray;

The fix:
Change private $rowColumnArray; to private $rowColumnArray = [];

==========

Issue 2:
After resolving the above, you will see a new issue.
Notice: Undefined offset: 1 in gf-form-multicolumn/includes/public/WH_GF_Multicolumn_Public_Form_Current.php on line 152

Caused by:

$divisor = ( $this->rowColumnArray[$this->rowCounter] > 0 ) ?
    $this->rowColumnArray[$this->rowCounter] : 1;

Why:
This assumes that $this->rowColumnArray has an index of $this->rowCounter but since $this->rowColumnArray starts as an empty array, it will result in an undefined index.

The fix:
Change

$divisor = ( $this->rowColumnArray[$this->rowCounter] > 0 ) ?
    $this->rowColumnArray[$this->rowCounter] : 1;

To

$divisor = ( ( $this->rowColumnArray[$this->rowCounter] ?? 0 ) > 0 ) ?
    $this->rowColumnArray[$this->rowCounter] : 1;

If the index does not exist, the value will default to 0.

All changes proposed are compatible with your plugins minimum PHP version, 7.0.

Thanks

• This reply was modified 3 years, 3 months ago by codepuncher.

Hi @webholism,

The division by zero issue is resolved, but the fix has resulted in a new issue:
[29-Jul-2021 09:26:19 UTC] PHP Notice: Trying to access array offset on value of type null in /path/to/site/plugin/folder/plugins/gf-form-multicolumn/includes/public/WH_GF_Multicolumn_Public_Form_Current.php on line 152

In the emails between us, I did share a bug fix with you for this.

Do you know when you can have that resolved?

Thanks

Thanks @webholism .

Do you have an ETA on when a new release will be published?

Hi @webholism,

I’ve sent an email to request the updated version.

Thanks

We are also experiencing this same issue on multiple websites and cannot see any type of pattern causing this.

It seems that this issue has been reported before: https://www.ads-software.com/support/topic/division-by-zero-error-in-gf-plugin/

What’s the best way to get this issue resolved?

PR is now merged.

Hi all,

I have submitted an issue on the GitHub repo regarding this.
https://github.com/timersys/wordpress-email-templates/issues/40

It seems that our PR https://github.com/timersys/wordpress-email-templates/pull/29 introduced a breaking change into the plugin.

We are using custom email templates (https://timersys.com/customize-wordpress-email-templates-plugin/) and because our templates used include_once instead of include, this is where the issue lies.

If you have any templates using include_once, simply change it to include and it will work.

I also have this issue, however, it is with Gravity Forms.

I’m currently trying to find if there is a way to use a filter to set or change it upon the event of the email being sent from Gravity Forms.

In my tests I can see that the 3rd line “return $email” in this method gets hit twice.

public function set_from_email( $email ){
	if( empty( $this->opts['from_email'] ) )
		return $email;
	return $this->opts['from_email'];
}

On the first time it’s hit, it shows the default from email of the plugin.
However, on the second time it’s hit it shows the from email that is set by Gravity Forms.

I’m not sure why it’s being run twice, but it looks like this could be solved once I’ve figured out why the first (incorrect) email address is being used and not the second.

• This reply was modified 6 years, 9 months ago by codepuncher.
• This reply was modified 6 years, 9 months ago by codepuncher.

Thanks for sorting this guys! Muchos appreciated