#34917 closed enhancement (duplicate)
Use meaningful variable names in wpLists
Reported by: | helgatheviking | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | has-patch needs-testing dev-feedback |
Focuses: | javascript, administration | Cc: |
Description
The wp-includes/js/wplists.js
script is cryptic and unnecessarily difficult to read. Single letters, such as e, s, r, f, a, don't tell us anything about what the variable is, or what type of data it is containing, which ultimately makes this script really difficult to work with.
Is there anybody who can translate these variables? At least for the non-minified version?
Attachments (1)
Change History (7)
#4
@
8 years ago
- Keywords has-patch needs-testing dev-feedback added; needs-patch removed
Here's a first draft, I renamed variables and added docblock to main functions.
next steps
- apply code format
- docblock anonymous functions
- describe function functionality
Note: See
TracTickets for help on using
tickets.
+1 for self-documenting code, Let's do it.