Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatibility issue Orbit Fox- Elementor Pro #844

Closed
AndreeaCristinaRadacina opened this issue Apr 15, 2024 · 4 comments · Fixed by #845
Closed

Compatibility issue Orbit Fox- Elementor Pro #844

AndreeaCristinaRadacina opened this issue Apr 15, 2024 · 4 comments · Fixed by #845
Assignees
Labels
bug This label could be used to identify issues that are caused by a defect in the product. customer report Indicates the request came from a customer. Priority-Medium Expected resolution time - up to 1 month. small (1-3h) This label is used for issues that can be completed within 3 hours or less.

Comments

@AndreeaCristinaRadacina
Copy link

AndreeaCristinaRadacina commented Apr 15, 2024

Description

Users reported incompatibility issues between Orbit Fox and Elementor Pro.

I was not able to replicate it.

Step-by-step reproduction instructions

  1. Install Elementor and Elementor Pro
  2. Install Orbit Fox
  3. Update WP to the latest version

Screenshots, screen recording, code snippet or Help Scout ticket

https://secure.helpscout.net/conversation/2569937337/413766/
elementor error
system-info-www.enablersofchange.com.au-15-04-2024.txt

Environment info

No response

Is the issue you are reporting a regression

Yes, this is a regression.

@AndreeaCristinaRadacina AndreeaCristinaRadacina added the bug This label could be used to identify issues that are caused by a defect in the product. label Apr 15, 2024
@AndreeaCristinaRadacina
Copy link
Author

@pirate-bot pirate-bot added customer report Indicates the request came from a customer. regression Issue represents a change in behavior from a previous version that is not intended or desired.. labels Apr 15, 2024
@vytisbulkevicius vytisbulkevicius removed the regression Issue represents a change in behavior from a previous version that is not intended or desired.. label Apr 15, 2024
@vytisbulkevicius
Copy link
Contributor

@AndreeaCristinaRadacina,

We haven't made a release for 3 weeks, so I wouldn't call this a regression. I see Elementor made a release today and both tickets are submitted today, so it seems like the issue was introduced by Elementor release.

Also, the error is also thrown by Elementor not OrbitFox:
image

@preda-bogdan, can you also take a look when you can to see if it's a problem that should be fixed from our end?

@preda-bogdan
Copy link
Contributor

preda-bogdan commented Apr 16, 2024

@vytisbulkevicius it is caused by https://github.com/Codeinwp/elementor-extra-widgets which is a dependency of OBFX and it will trigger and error inside the new version of Elementor Pro because of this line here:https://github.com/Codeinwp/elementor-extra-widgets/blob/master/class-elementor-extra-widgets.php#L139 this should return the $data instead of NULL to not crash the editor on the newer versions od Elementor PRO.
You can assign this to me and I can take care of resolving this.

@vytisbulkevicius vytisbulkevicius added the Priority-Medium Expected resolution time - up to 1 month. label Apr 16, 2024
@preda-bogdan preda-bogdan added the small (1-3h) This label is used for issues that can be completed within 3 hours or less. label Apr 16, 2024
preda-bogdan added a commit to Codeinwp/elementor-extra-widgets that referenced this issue Apr 16, 2024
@preda-bogdan preda-bogdan linked a pull request Apr 16, 2024 that will close this issue
@preda-bogdan
Copy link
Contributor

I've patched the issue in the linked package, updated the dependency, and prepared a release.
I've skipped the code review as the change was very small.
cc: @vytisbulkevicius
Do you want me to do a release?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This label could be used to identify issues that are caused by a defect in the product. customer report Indicates the request came from a customer. Priority-Medium Expected resolution time - up to 1 month. small (1-3h) This label is used for issues that can be completed within 3 hours or less.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants