SQL reports are broken. Here is a patch that fixes.

CommentFileSizeAuthor
#3 patch_31.patch916 bytesgábor hojtsy
#2 patch.patch765 byteswebernet
#1 patch.patch709 byteswebernet
system.admin_.inc-sql.patch598 bytescwgordon7

Comments

webernet’s picture

webernet commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new patch.patch709 bytes

Whoops. This was broken in a code style patch.

Rerolled to include a comment that the title case variable names are required so this doesn't happen again.

webernet’s picture

webernet commented
StatusFileSize
new patch.patch765 bytes

Better comment.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed
StatusFileSize
new patch_31.patch916 bytes

Well, I tried to look up "title case", and most I got was contradictory information. Wikipedia says it can mean various number of things depending on style. So I made that part much more to the ground. Also noticed that this function had a comment about it being a PHP status page, which was a copy-paste error. Committed this.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.