[wp-trac] [WordPress Trac] #40655: DB error during installation
WordPress Trac
noreply at wordpress.org
Tue Sep 5 07:52:40 UTC 2017
#40655: DB error during installation
--------------------------+------------------
Reporter: FlyerUA | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.9
Component: Database | Version: 4.6
Severity: normal | Resolution:
Keywords: | Focuses:
--------------------------+------------------
Comment (by danieltj):
Replying to [comment:4 SergeyBiryukov]:
> Replying to [comment:3 danieltj]:
> > I'd suggest that we just make it more user friendly like the other
error messages. I think what it's doing is fine because it'll error anyway
when using an invalid prefix so the idea is we have the user change it at
some point to something valid.
>
> I don't mind making the `"Table Prefix" is invalid` message more
friendly, but that's not the issue here, `wp_` is a perfectly valid
prefix.
>
> The check added in [37581] is supposed to disallow prefixes like `7e1_`
that could be parsed as a value. This appears to work as expected, but in
some cases it triggers the `Unknown column 'wp_' in 'field list' for query
SELECT wp_` error, which should be silently discarded, not displayed to
the user.
Ahh I see. I'll have to revisit this then because I couldn't replicate the
`wp_` prefix error, only with the `7e1_` prefix.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/40655#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list