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

1.31.1 Release #1539

Closed
wants to merge 12 commits into from
Closed

1.31.1 Release #1539

wants to merge 12 commits into from

Conversation

jom
Copy link
Member

@jom jom commented Jul 4, 2018

This release is slated for Wednesday, 2018-07-11 (pending major issues).

Download (Beta 2) | PRs | Issues | Diff

Overview and Testing

While this release is small in terms of functionality impact, it is massive in terms of lines of modified/added/removed code. We'll be heavily testing this release, however, this should be backwards compatible with 1.31.0. Still, we highly recommend theme and plugin authors test their product with this beta release. We had two massive PRs in this release:

  • Adds additional escaping throughout plugin (VIP) #1495: This PR added escaping where it was missing in the plugin. Many of these are with string translations or output that shouldn't be affected by themes/plugins, but please test to make sure you don't see escaped HTML anywhere it should be. Thanks to @tomjn for all his work on this.
  • PHPCS code formatting fixes #1534: This PR worked on cleaning up our PHPCS violations, including:
    • Adding some additional escaping.
    • Remove all unnecessary uses of extract(). Themes and plugins that override the templates related to [jobs] should verify their products aren't effected by the change.
    • Added translation hints where there were placeholders in translated strings.
    • Whitespace fixes.
    • Logic/yoda swaps.
    • Shift to always use strict comparisons with ===, !==, and in_array( $needle, $haystack, true ).
    • All devs should do a composer install on master branch to install the pre-commit hook that will prevent further PHPCS issues from entering codebase. Some ignored rules will be handled soon.

Additionally, the following PRs should be tested using the testing instructions provided inside the PR or related issue:

Changelog

= 1.31.1 =
* Enhancement: Add option to show company logo in Recent Jobs widget. (@RajeebTheGreat)
* Enhancement: Suggest additional cookie information on Privacy Policy page.
* Enhancement: Add WPJM related meta data to user data extract.
* Fix: Tightened the security of the plugin with additional string escaping.
* Fix: Issue with map link in admin backend. (@RajeebTheGreat)
* Fix: No longer auto-expire job listings in Draft status.
* Fix: Issue with undefined index error in WP admin. (@albionselimaj)
* Fix: Issue with duplicate usernames preventing submission of job listings. (@timothyjensen)
* Dev: Widespread code formatting cleanup throughout the plugin. 

Known Issues

Beta 2 (2018-07-05)

Diff from Beta 1

  • @tripflex's field editor allows HTML in field labels. Switch field label output escaping to wp_kses_post() instead of esc_html(). (b55a749)
  • Duplicate post issue caused by a merge/rebase issue. (382a5a0)

Beta 1 (2018-07-04)

  • Escaped checkbox labels in admin settings
  • Removes embedded company videos
  • Removes HTML that should be allowed from some calls to wpjm_get_the_job_title()
  • templates/content-widget-job_listing.php needs to check for $show_logo because it is shared in the widgets (we should probably add this to the other widget)

Additional Information

This release PR should never be merged. Instead, once completed, we'll tag the release and then cherry-pick the updates to the version number into a clean PR along with additional fixes. Since branching it from master, I've reverted the changes related to the upcoming blocks (as none will be there in this release).

@jom jom changed the title [WIP] 1.31.1 Release 1.31.1 Release Jul 4, 2018
@jom jom force-pushed the release/1.31.1 branch from bbd5518 to 64cf405 Compare July 11, 2018 16:28
@jom jom mentioned this pull request Jul 11, 2018
@jom jom closed this Jul 11, 2018
@jom jom deleted the release/1.31.1 branch February 18, 2019 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant