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

fix: resolve potential issues with new synonymous for GeometryCollection type #8927

Merged
merged 1 commit into from
Aug 24, 2022
Merged

fix: resolve potential issues with new synonymous for GeometryCollection type #8927

merged 1 commit into from
Aug 24, 2022

Conversation

davidsl4
Copy link
Contributor

@davidsl4 davidsl4 commented Apr 23, 2022

Description of change

As of MySQL v8.0, a new synonymous was introduced for "GeometryCollection" - "GeomCollection".
The preferred type name is "GeomCollection".
This fix will change the synonym name to its full name.

Pull-Request Checklist

  • Code is up-to-date with the master branch
  • npm run format to apply prettier formatting
  • npm run test passes with this change
  • This pull request links relevant issues as Fixes #0000
  • There are new or updated unit tests validating the change N/A
  • Documentation has been updated to reflect this change N/A
  • The new commits follow conventions explained in CONTRIBUTING.md

As of mysql v8.0, new synonymous was introduced for "GeometryCollection" - "GeomCollection".
The preferred type name is "GeomCollection".
@davidsl4 davidsl4 changed the title fix: resolve issues with new synonymous for GeometryCollection type fix: resolve potential issues with new synonymous for GeometryCollection type Apr 23, 2022
@AlexMesser
Copy link
Collaborator

Can you please add a test case that shows the problem you are trying to solve?

@davidsl4
Copy link
Contributor Author

davidsl4 commented Apr 29, 2022

Sure! On my machine it fails the following test case
image
Here is how it stored in the database
image

table!
.findColumnByName("geometrycollection")!
.type.should.be.equal("geometrycollection")

@AlexMesser AlexMesser merged commit 5ebc626 into typeorm:master Aug 24, 2022
@AlexMesser
Copy link
Collaborator

thank you for contribution!

wirekang pushed a commit to wirekang/typeorm that referenced this pull request Aug 25, 2022
…n MySQL 8 (typeorm#8927)

As of mysql v8.0, new synonymous was introduced for "GeometryCollection" - "GeomCollection".
The preferred type name is "GeomCollection".
nordinh pushed a commit to nordinh/typeorm that referenced this pull request Aug 29, 2022
…n MySQL 8 (typeorm#8927)

As of mysql v8.0, new synonymous was introduced for "GeometryCollection" - "GeomCollection".
The preferred type name is "GeomCollection".
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

Successfully merging this pull request may close these issues.

None yet

2 participants