Make WordPress Core

Opened 15 years ago

Closed 11 years ago

Last modified 11 years ago

#14657 closed enhancement (duplicate)

Resync Boolean Type Case

Reported by: jacobsantos's profile jacobsantos Owned by:
Milestone: Priority: normal
Severity: trivial Version:
Component: General Keywords: has-patch needs-refresh
Focuses: Cc:

Description

TRUE to true and FALSE to false.

It seems patches weren't inline with the lowercase usage of the boolean type. Depends on programmer preference.

Attachments (1)

booleans_are_types.patch (42.2 KB) - added by jacobsantos 15 years ago.
Change case of booleans in files.

Download all attachments as: .zip

Change History (7)

@jacobsantos
15 years ago

Change case of booleans in files.

#1 @hakre
15 years ago

From what I remember, the use of the constants true and false should be lowercase. Patch looks good in that sense to me.

#2 @jacobsantos
15 years ago

Well, to me they should be lowercase because of having to type them out a lot. The other reason is that language reserve words or keywords should always be lowercase to differentiate from other user defined terms.

The argument is whether or not the language type lowercase usage should be used verses the language implementation of them actually being constants, albeit case-insensitive constants.

The problem also is that if we switch over to using what is defined as constants, we'll have to use NULL instead of null as well.

#3 @nacin
15 years ago

  • Keywords needs-refresh added; tested removed
  • Milestone changed from Awaiting Review to Future Release

No way this patch still applies.

This patches a few external libraries. We shouldn't patch those, it becomes more difficult to merge in changes from upstream later on.

#4 @jacobsantos
13 years ago

  • Status changed from new to assigned

#5 @wonderboymusic
11 years ago

  • Milestone Future Release deleted
  • Resolution set to wontfix
  • Status changed from assigned to closed

FALSE is in _http_build_query() once, but that's it. The rest are in external libraries.

#7 @SergeyBiryukov
11 years ago

  • Resolution changed from wontfix to duplicate

Duplicate of #16302.

Note: See TracTickets for help on using tickets.