Changeset 1602 for sites/trunk/jobs.wordpress.net/public_html/wp-content/plugins/jobswp/jobswp.php

Timestamp:

05/20/2015 05:57:13 PM (11 years ago)

Author:

coffee2code

Message:

jobs.wordpress.net: Explicitly declare class methods as public or protected. Fix a static method invocation.

File:

• sites/trunk/jobs.wordpress.net/public_html/wp-content/plugins/jobswp/jobswp.php (modified) (22 diffs)

sites/trunk/jobs.wordpress.net/public_html/wp-content/plugins/jobswp/jobswp.php

@@ -70 +70 @@
          * Constructor
          */
-        function __construct() {
+        protected function __construct() {
                 if ( class_exists( 'Walker_Jobs_Category' ) )
                         self::$walker = new Walker_Jobs_Category;
@@ -83 +83 @@
          * Invokes registration of custom post type, taxonomy, and statuses.
          */
-        function registrations() {
+        public function registrations() {
                 $this->register_post_type();
                 $this->register_post_status();
@@ -93 +93 @@
          * as any processing that needs to happen directly on 'init'.
          */
-        function init() {
+        public function init() {
                 // Allow customization of the number of days until a job gets pruned.
                 // By default, it is 21 days.
@@ -129 +129 @@
          * @return array The amended list of allowed tags
          */
-        function wp_kses_allowed_html( $allowedtags, $content ) {
+        public function wp_kses_allowed_html( $allowedtags, $content ) {
                 // Add permissable tags
                 $allowedtags['ol'] = array();
@@ -151 +151 @@
          * @return array The amended list of classes to be added to 'body' tag
          */
-        function body_class( $classes ) {
+        public function body_class( $classes ) {
                 $classes[] = 'jobswp';
                 return $classes;
@@ -165 +165 @@
          * @return array Amended associated array of column names and labels
          */
-        function posts_columns( $columns, $post_type ) {
+        public function posts_columns( $columns, $post_type ) {
                 if ( 'job' !== $post_type ) {
                         return $columns;
@@ -180 +180 @@
          * @param int $post_id The post ID
          */
-        function custom_posts_columns( $column_name, $post_id ) {
+        public function custom_posts_columns( $column_name, $post_id ) {
                 switch ( $column_name ) {
                         case 'poster':
@@ -200 +200 @@
          * Registers the custom post type.
          */
-        function register_post_type() {
+        protected function register_post_type() {
                 $labels = array(
                         'name'                => _x( 'Jobs', 'Post Type General Name', 'jobswp' ),
@@ -242 +242 @@
          * Registers the custom post statuses.
          */
-        function register_post_status() {
+        protected function register_post_status() {
                 // Status of 'closed' indicates a job that was unpublished from the site.
                 register_post_status( 'closed', array(
@@ -268 +268 @@
          * Registers the custom taxonomy.
          */
-        function register_taxonomy()  {
+        protected function register_taxonomy()  {
                 $labels = array(
                         'name'                       => _x( 'Job Categories', 'Taxonomy General Name', 'jobswp' ),
@@ -301 +301 @@
          * user account does not exist.
          */
-        function alert_if_no_jobposter() {
+        public function alert_if_no_jobposter() {
                 global $pagenow;
 
@@ -319 +319 @@
          * Outputs button to close a job on the post edit page.
          */
-        function post_submitbox_start() {
+        public function post_submitbox_start() {
                 global $post;
 
@@ -365 +365 @@
          * @return boolean True == the job can be closed
          */
-        function can_job_be_closed( $post ) {
+        protected function can_job_be_closed( $post ) {
                 // The post must exist
                 if ( ! $post )
@@ -468 +468 @@
          * @return array
          */
-        function post_row_actions( $actions, $post ) {
+        public function post_row_actions( $actions, $post ) {
                 if ( $this->can_job_be_closed( $post ) )
                         $actions['close'] = $this->_get_close_link( $post );
@@ -483 +483 @@
          * @return string The fixed text
          */
-        function WordPress_dangit( $text ) {
+        public function WordPress_dangit( $text ) {
                 return str_replace(
                         array( 'Wordpress', 'wordpress', 'wordPress', 'word press', 'Word press', 'word Press', 'Word Press' ),
@@ -498 +498 @@
          * @return string The content appended with the post-a-job form
          */
-        function add_post_a_job_form( $content ) {
+        public function add_post_a_job_form( $content ) {
                 if ( ! $this->skip_content && is_page( 'post-a-job' ) ) {
                         $this->skip_content = true;
@@ -517 +517 @@
          * unverified visitor.
          */
-        function save_job() {
+        public function save_job() {
                 if ( isset( $_POST['postjob'] ) && 1 == $_POST['postjob'] ) {
                         check_admin_referer( 'jobswppostjob' );
@@ -551 +551 @@
                         // If everything checks out, create the job
                         if ( $this->success ) {
-
                                 $job_id = $this->create_job();
-
                                 if ( is_wp_error( $job_id ) ) {
                                         $_POST['errors'] = $job_id->get_error_message();
@@ -606 +604 @@
 
                                 // Massage and sanitize the field value depending on field
-                                $val = $this->validate_job_field( $field, $_POST[ $field ], $_POST );
+                                $val = self::validate_job_field( $field, $_POST[ $field ], $_POST );
 
                                 add_post_meta( $job_id, $field, $val );
@@ -627 +625 @@
          * Prunes old jobs.
          */
-        function scheduled_job_pruning() {
+        public function scheduled_job_pruning() {
                 global $wpdb;
 
@@ -641 +639 @@
          * Unschedules job pruning cron.
          */
-        static function unschedule_job_pruning() {
+        public static function unschedule_job_pruning() {
                 wp_clear_scheduled_hook( 'jobswp_scheduled_job_pruning' );
         }
@@ -658 +656 @@
          * @return string
          */
-        function validate_job_field( $field, $value, $extra_data = array() ) {
+        public static function validate_job_field( $field, $value, $extra_data = array() ) {
                 switch ( $field ) {
                         case 'email':