-
Notifications
You must be signed in to change notification settings - Fork 367
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PHPCS code formatting fixes #1534
Merged
+5,280
−3,797
Merged
Changes from 1 commit
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
3afdc61
Add PHPCompatibility PHPCS rules
jom 9863c54
Change PHPCS to more closely match Woo
jom a5e3ae9
PHPCS: Auto formatting fixes for all PHP files
jom 3326012
PHPCS: Add periods at end of single line comments
jom d817576
PHPCS: Remove all VIP rules
jom 8bace7d
PHPCS: Add periods at end of param comments
jom 5204c68
PHPCS: Add full stop at end of return comments
jom 44a0694
PHPCS: Remove some rules that aren't a priority right now
jom ebaafc9
PHPCS: Fix some strange auto-fixes
jom d9259fe
PHPCS: Remove usage of `extract()` in shortcodes
jom a565086
PHPCS: PHP DocBlock fixes for return types and thrown items
jom ef931d8
Remove duplicate keys from usage data
jom 999c9c3
Code Smell: Use parent constructor in widgets
jom 59d2400
Bug: Don't reuse same var names in child foreach loop
jom 6688a30
Merge branch 'master' into change/phpcs-fixes
jom 771fbd2
PHPCS: Formatting fix from master merge of wp-job-manager.php
jom 37a2b65
PHPCS: Expanded method args with array, start array new declaration o…
jom 56abbe7
PHPCS: Add auto formatting to tests
jom c7e8f00
PHPCS: Auto fixes for tests
jom 1ae3420
PHPCS: Yoda, docblock, some translation comments
jom 73ec587
PHPCS: 3rd party code cleanup
jom 51ecd6a
PHPCS: Abstract class documentaiton cleanup
jom 4b30ea0
PHPCS: Escape LIKE replacement vars correctly in admin CPT
jom c13231e
PHPCS: Admin code fixes
jom b35d776
PHPCS: Expand and ignore XSS on pre-escaped setting fields
jom 6d914da
PHPCS: Minor formatting and translation fixes for email notifications
jom a0d23bc
PHPCS: Minor fixes for helper and form classes
jom 1b9f6d1
PHPCS: Mainly docblock fixes for REST API classes
jom 11b1dee
PHPCS: Minor fixes to widget formatting and translation hints
jom b51a263
PHPCS: Minor fixes and docblock changes for root includes files
jom d0a477e
PHPCS: Adds additional escaping, formatting, and minor doc changes to…
jom acde6aa
PHPCS: Formatting, escaping, and translator hints for post type class
jom fbde49a
PHPCS: Minor rule ignoring for category walker
jom 19bd42a
PHPCS: Update yoda and remove inside var setting in deprecated functions
jom c5a2e86
PHPCS: Update core functions
jom 2b3ea96
PHPCS: Fix issues with template functions
jom 7b03665
PHPCS: Fix issues with core WP_Job_Manager class
jom 4194575
PHPCS: Add some globally ignored rules
jom 8b8b4ad
PHPCS/Tests: Update tests for updated email functions
jom e8aee37
PHPCS: Revert some changes in cache helper and cpt
jom d0c43d6
PHPCS: Revert change to not return when echo-ing in template functions
jom 7aaf50e
Update renamed callbacks for sending notifications
jom 62d06b7
PHPCS: Revert change on settings for page input
jom 7b0d873
PHPCS: Fix some whitespace autofix errors in writepanels
jom 15330f7
PHPCS: Add automatic PHPCS checking on pre-commit hook. Mu-ha-ha-ha.
jom 2d58bb7
PHPCS: Escape translation in settings file
jom e058f5a
PHPCS: Collapse some auto-changes for if statements
jom cc670d7
PHPCS: Remove unnecessary `? true : false` casting
jom 91fee33
PHPCS: Format SQL prepare statements in post types
jom bd79c18
PHPCS: Fix issue with auto formatting in html license view
jom 7658c29
PHPCS: Don't escape before/after widget and before/after title
jom c7735d5
PHPCS: Fix issues with changed form validation method
jom a3b335e
PHPCS: Fix minor issue in addons file
jom 2a77dfe
PHPCS: Fix minor issue in admin file
jom d110743
PHPCS: Minor code formatting fixes
jom 235549c
Remove redundant response check in helper
jom 7d2a6f5
PHPCS: Add explicit rule exception for wp_new_user_notification
jom 847b692
PHPCS: Revert change to selected_category attribute in `[jobs]`
jom File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
PHPCS: Fix issues with changed form validation method
- Loading branch information
commit c7735d5fe25e6b95631dd01e19e7c260c9cd98b4
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This logic is no longer correct. A non-JSON error will not return a
WP_Error
here.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in c7735d5