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]: problem with adding new table via plugin sql table #5244

Open
Jimmi08 opened this issue May 1, 2024 · 0 comments
Open

[Bug]: problem with adding new table via plugin sql table #5244

Jimmi08 opened this issue May 1, 2024 · 0 comments
Labels
type: bug A problem that should not be happening
Milestone

Comments

@Jimmi08
Copy link
Contributor

Jimmi08 commented May 1, 2024

What e107 version are you using?

Latest Github version (just updated)

Bug description

Maybe this worked this way before

I have plugin, I need to add new table. I added it to SQL file:

CREATE TABLE `race_result` (
`race_result_id` int(10) NOT NULL AUTO_INCREMENT,
`race_result_racer_id` varchar(100) NOT NULL,
`race_result_racer_number` VARCHAR(11) NOT NULL,
`race_result_time` varchar(255) NOT NULL,
`race_result_sent` INT(10) NOT NULL,
`race_result_log` LONGTEXT NOT NULL,
`race_result_created` INT(10) NOT NULL,
`race_result_updated` INT(10) NOT NULL,
PRIMARY KEY (`race_result_id`)
) ENGINE=InnoDB;

e107_update detects this correctly but after the update action:

Update Unsuccessful [CREATE TABLE e107_race_result () ENGINE=InnoDB DEFAULT CHARACTER SET=utf8mb4;]

Notice missing fields.

Tools/Database check and run fix from there solved this, but there is little issue with not refreshing after running fix and it looks like not working, only if you run it again, you will get message that table already exists.

How to reproduce

No idea, use that code?

Expected behavior

Added table to database in e107 update functionality

What browser(s) are you seeing the problem on?

Firefox, Chrome / Brave

PHP Version

PHP 7.4

@Jimmi08 Jimmi08 added the type: bug A problem that should not be happening label May 1, 2024
@CaMer0n CaMer0n added this to the e107 2.4.0 milestone May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A problem that should not be happening
Projects
None yet
Development

No branches or pull requests

2 participants