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.

Sign up for GitHub

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

Jump to bottom

Bug rename table during migration #322

Closed
alex8276 opened this issue Jul 12, 2019 · 2 comments · Fixed by #323
Closed

Bug rename table during migration #322

alex8276 opened this issue Jul 12, 2019 · 2 comments · Fixed by #323
Labels
bug
Milestone
1.10.1

Comments

@alex8276
Copy link

alex8276 commented Jul 12, 2019 •
edited
Loading

Hello,

During migration 1.7.3 to 1.10 in fixfieldsname function, the request $bad_named_fields give some bad results. So many tables are renamed by error because of Mysql use POSIX 1003 standard and it doesn't \d.

Regards
@tomolimo
Copy link
Collaborator

Hello,
Which version of mySQL are you using?
Thank you,
Regards,
Tomolimo

@alex8276
Copy link
Author

I use the version : 5.7.17

regards

tomolimo added a commit to tomolimo/fields that referenced this issue Jul 12, 2019
@tomolimo
Changed \d by [0-9] in regex for mysql, as \d is not POSIX
17f50df 
fixes pluginsGLPI#322
@tomolimo tomolimo mentioned this issue Jul 12, 2019
Merged
cedric-anne pushed a commit to cedric-anne/glpi-plugin-fields that referenced this issue Jul 15, 2019
@tomolimo @cedric-anne
Changed \d by [0-9] in regex for mysql, as \d is not POSIX
f254b12 
fixes pluginsGLPI#322
@cedric-anne cedric-anne added this to the 1.10.1 milestone Jul 15, 2019
cedric-anne pushed a commit to cedric-anne/glpi-plugin-fields that referenced this issue Jul 15, 2019
@tomolimo @cedric-anne
Changed \d by [0-9] in regex for mysql, as \d is not POSIX
5b174da 
fixes pluginsGLPI#322
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
1.10.1
Development

Successfully merging a pull request may close this issue.

Changed \d by [0-9] in regex for mysql, as \d is not POSIX tomolimo/fields
3 participants
@tomolimo @alex8276 @cedric-anne