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

[Bug] When using console command generate:dimension it always generates column type of INTEGER(11) DEFAULT 0 NULL #22908

Closed
4 tasks done
mikkeschiren opened this issue Jan 6, 2025 · 5 comments · Fixed by #22909
Labels
Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. To Triage An issue awaiting triage by a Matomo core team member

Comments

@mikkeschiren
Copy link
Contributor

What happened?

When using the console command to generate a new dimension, in this case a visit dimension, and you provide any valid input of column type, it always ends up beeing INTEGER(11) DEFAULT 0 NULL in the generated dimension.

What should happen?

The generate command should output the column type provided.

How can this be reproduced?

Run ./console generate:dimension (first I have created a dummy plugin). When providing the input of column type, it ends out always beeing INTEGER(11) DEFAULT 0 NULL.

Matomo version

5.2.1

PHP version

8.2.27

Server operating system

Linux

What browsers are you seeing the problem on?

No response

Computer operating system

No response

Relevant log output

No response

Validations

@mikkeschiren mikkeschiren added Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. To Triage An issue awaiting triage by a Matomo core team member labels Jan 6, 2025
@mikkeschiren
Copy link
Contributor Author

If you do create an action dimension - you get the column type you asked for.

@mikkeschiren
Copy link
Contributor Author

It seems that the string replacement is wrong, in the generate command it looks for INTEGER(11) DEFAULT 0 NOT NULL in the ExampleTracker it is INTEGER(11) DEFAULT 0 NULL.

@mikkeschiren
Copy link
Contributor Author

Suggested fix: #22909

@sgiehl sgiehl linked a pull request Jan 7, 2025 that will close this issue
11 tasks
@sgiehl
Copy link
Member

sgiehl commented Jan 7, 2025

fixed with #22909

@sgiehl sgiehl closed this as completed Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. To Triage An issue awaiting triage by a Matomo core team member
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants