Skip to content
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

WPCS Compliance, GitIgnore, Security #10

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

smileBeda
Copy link

Pull Request

What changed?

  • [Added] gitIgnore
  • [Added] WPCS ignore rules where applicable
  • [Changed] Renamed CamelCase to wpcs compliant lowercase with underscores, as well as conflicting global names renamed to "local" names
  • [Changed] Applied WPCS as widely as possible
  • [Fixed] A few instances of unescaped echoed contents
  • [Fixed] A few instances of unsanitized User Input

Why did it change?

  • Security enhanced
  • Cross-Developer and platform ease to edit and contribute when having applied coding standards and an available (and of course extendable) gitignore. All over code readability improved.
  • Avoid conflicts with global namespaces

Did you fix any specific issues?

  • Missing escapes and sanitizes on some instances could be considered Fixes.

NOTE
No functionality should have changed or been broken, yet, it is recommended to first test this on a staging copy of the site to avoid any issues in the open.

CERTIFICATION

By opening this pull request, I do agree to abide by the CODE OF CONDUCT and be bound by the terms
of the Contribution Guidelines in effect on the date and time of my contribution as proven by the
revision information in GitHub. I also agree that any previous contributions shall be deemed subject to the terms of the
version in effect on the date and time of this pull request, or any future revisions for pull requests I may submit.

Further, I certify that this work is my own, is original, does not violate the intellectual property of any other person
or entity, and I am not violating any license agreements or contracts I have with any person or entity.

Finally, I agree that this code may be licensed under any license deemed appropraite by AspirePress, including but not
limited to open source, closed source, proprietary or custom licenses, and that such license terms neither violate my
rights or my copyright to this code.

(Please check a box below)

[x] I agree
[ ] I do not agree

- [Added] gitIgnore
- [Added] WPCS ignore rules where applicable
- [Changed] Renamed CamelCase to wpcs compliant lowercase with underscores, as well as conflicting global names renamed to "local" names
- [Changed] Applied WPCS as widely as possible
- [Fixed] A few instances of unescaped echoed contents
- [Fixed] A few instances of unsanitized User Input
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant