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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webernet’s picture

webernet CreditAttribution: webernet commented
Status: Needs review » Reviewed & tested by the community
FileSize
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 CreditAttribution: webernet commented
FileSize
patch.patch765 bytes

Better comment.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Reviewed & tested by the community » Fixed
FileSize
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) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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