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

The behavior of the modules fundamentally change in >OXID v.6.2 #5

Open
TumTum opened this issue Feb 8, 2022 · 5 comments
Open

The behavior of the modules fundamentally change in >OXID v.6.2 #5

TumTum opened this issue Feb 8, 2022 · 5 comments
Assignees

Comments

@TumTum
Copy link

TumTum commented Feb 8, 2022

  • If you save a variable in the backend. The module will be disabled / enabled
  • With the command oe-console oe:module:apply-configuration. The module will be disabled / enabled
    • If you have an EE version it will be disabled / enabled per store.

Knowing this, it means that the Events::onActivate() SQL will be executed, and in case of Events:: onDeactivate() tables will be deleted.

We have 9 SubShops with command the oe-console oe:module:apply-configuration.
We will delete the table 9 times and create it again. That means the start page of the store will throw a fatal error as soon as wleWallee_cron is deleted. Or just because you have changed a setting in the backend.

The Event modules are not suitable at all. For the first installation. There rather uses Doctrine module-migration

@sbossert sbossert self-assigned this Feb 8, 2022
@theodorspringwallee
Copy link
Contributor

Dear TumTum

Sorry for the late response. According to our devs this was not happening again.

Therefore, I will close this ticket.

@TumTum
Copy link
Author

TumTum commented Oct 6, 2022

Hi @theodorspringwallee

According to our devs this was not happening again.

What does that mean?
Then you have not understood the problem whatsoever!
These event classes slow down the whole system.

@theodorspringwallee
Copy link
Contributor

Dear TumTum

Sorry, I guess there was a misunderstanding about this issue. Our developers will run now a more in-depth investigation.

@theodorspringwallee
Copy link
Contributor

Dear TumTum

Our developer told me, that we currently can't reproduce your issue on OXID 6.10, where we currently support this.

Are you using an older OXID version?

Best regards

@TumTum
Copy link
Author

TumTum commented Nov 7, 2022

Very sad. Also in this OXID-CE 6.10 the problem is included. I don't want to explain this to you anymore.
I totally don't care now either. We do not use your module for these reasons.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants