doublesecretagency / craft-googlemaps

Google Maps plugin for Craft CMS - Maps in minutes. Powered by the Google Maps API.
https://plugins.doublesecretagency.com/google-maps/
Other
10 stars 8 forks source link

Updating from 4.2.5 -> 4.3.0 causes db error. #84

Closed delux589 closed 1 year ago

delux589 commented 1 year ago

We are trying to update all our craft plugins and we are hitting a problem when updating this plugin.

We are currently on version 4.2.5 but when updating to the latest version we then get a missing db column as in the screenshot below.

Screenshot 2023-05-22 at 13 14 59

this seems to be an issue after this migration has run when updating to the next available version which is 4.3.0 from our current version.

We have also tried updating all the way to 4.3.3 and the issue persists.

We are on the latest craft version.

I hope this helps.

lindseydiloreto commented 1 year ago

Hi @delux589, it's hard to say what the issue is here. Nothing in that stack trace is explicitly pointing at the Google Maps plugin.

Did you have to restart the update process at some point? If so, did you completely rollback the database first? It almost sounds like the system is trying to run an update after a previously failed migration.

The stack trace indicates that the process moves through your sitemodule module, specifically ImportFormattingService.php. You may want to check the custom code you have in there.

If you're confident that this is an issue with the Google Maps plugin, feel free to hit me up on Discord and I can help you work through it.

lindseydiloreto commented 1 year ago

Closing this thread, but feel free to ping me on Discord if you think you're still having a Google Maps issue. 👍

Kingo4luv commented 1 year ago

Hi @lindseydiloreto

I am following up on a previously closed issue (Issue #84) reported by my colleague. We are still encountering difficulties when updating the Google Maps plugin from version 4.2.5 to 4.3.0​1​.

Based on our investigations, it appears that the migration associated with this update is causing a column (field_address) to be removed from the database. This seems to be the root cause of the error we are facing, as this column is required by our system.

Our findings are based on the following steps:

The error message we encounter when trying to use the system after the update specifically mentions the absence of the field_address column​.

We wanted to bring this to your attention as this might be an unintended side effect of the migration process. Any guidance on how to address this issue would be greatly appreciated.

lindseydiloreto commented 1 year ago

Hi @Kingo4luv, happy to help. Would you mind pinging me on Discord (lindseydiloreto#3339) so we can go into greater detail about your migration issue?

For various reasons (including complexity and security), it is much easier to troubleshoot migration issues via Discord.