RESOLVING + RESOLVING2 constants (Calculation/Calculation.php:4148)

[otuhacek]

Hi,
I just went into some problem when exporting some content on AC99 field. It complains of redefining RESOLVING2 constant multiple times, which triggered PHP Notice. Is this piece of code meant to be there or is it just something from previous testing?

PhpSpreadsheet/src/PhpSpreadsheet/Calculation/Calculation.php

Line 4148 in a972943

define('RESOLVING2', true);

Thanks

[pedrofsantoscom]

I also have the same notice. Besides the notice it runs without a problem.
This happens on a file created that has 22-29 sheets, and on each of those sheets column A has ~1000 rows and each row on col A has a very long expression (7000 characters long).

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If this is still an issue for you, please try to help by debugging it further and sharing your results.
Thank you for your contributions.

[jonnylink]

This is still an issue

[jonnylink]

This is from this commit: https://github.com/jonnylink/PhpSpreadsheet/commit/0b387e767e99a9042886ea673d9b9905c7951e87

It does in fact look like code that was simply there to test, since there are no other references to the constant RESOLVING or RESOLVING2 in the codebase. I'm going to make a PR to remove that broken code.

[Benjiiii]

Facing same issue

[woganmay]

Just had this come up in production now for the first time. Definitely still an issue.

[jonnylink]

Might want to comment on or thumbs up the PR (#1328) if you're having this issue. Not sure anyone is checking closed issues.

@PowerKiKi please let me know if there's anything else needed to help get this bug fix merged.

[oleibman]

Fixed by PR #1328 in February 2020.