Get WordPress

Make WordPress Core

Opened 4 months ago

Closed 4 months ago

#60813 closed defect (bug) (invalid)

MariaDB generating error when we are using %s as placeholder for table name when CS(code sniffer) suggest to used %s instead %i

Reported by: chiragrathod103's profile chiragrathod103 Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Database Keywords: reporter-feedback close
Focuses: Cc:

Description

ERROR WordPress.DB.PreparedSQLPlaceholders.UnsupportedIdentifierPlaceholder

The %i modifier is only supported in WP 6.2 or higher. Found: "%i".

Change History (3)

#1 @swissspidy
4 months ago

  • Keywords reporter-feedback close added

Where is this error being reported? In WordPress core?

If not, you should report such things to the affected plugin or theme. If this happens in your own project, the error message is pretty self-explanatory. "%i" is only supported on WP 6.2+, so if you support older versions you need to fall back to something else.

And if you believe there is an issue with the sniff itself, you should report it at ​https://github.com/WordPress/WordPress-Coding-Standards

#2 @chiragrathod103
4 months ago

@swissspidy Thank you for the clarify. I will move a ticket there.

#3 @swissspidy
4 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

Well you didn't really answer the questions, and just opening a ticket there without answering those questions or providing additional context is not really helpful or productive. Good luck finding a solution though

Note: See TracTickets for help on using tickets.