Changeset 2053454

Timestamp:

03/19/2019 02:21:23 PM (7 years ago)

Author:

windyjonas

Message:

Version bump and code cleanup

Location:

user-activity/trunk

Files:

• classes/class-bu-user-activity-table.php (modified) (4 diffs)
• readme.txt (modified) (3 diffs)
• user-activity.php (modified) (1 diff)

user-activity/trunk/classes/class-bu-user-activity-table.php

@@ -101 +101 @@
          * Handle request parameters
          */
-        $ptype     = ( isset( $_REQUEST['ptype'] ) && ! empty( $_REQUEST['ptype'] ) ? $_REQUEST['ptype'] : 'all' );
-        $startdate = ( isset( $_REQUEST['startdate'] ) ? $_REQUEST['startdate'] : '' );
-        $enddate   = ( isset( $_REQUEST['enddate'] ) ? $_REQUEST['enddate'] : '' );
-        $username  = ( isset( $_REQUEST['s'] ) ? $_REQUEST['s'] : '' );
+        $ptype     = ( isset( $_REQUEST['ptype'] ) && ! empty( $_REQUEST['ptype'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['ptype'] ) ) : 'all' );
+        $startdate = ( isset( $_REQUEST['startdate'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['startdate'] ) ) : '' );
+        $enddate   = ( isset( $_REQUEST['enddate'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['enddate'] ) ) : '' );
+        $username  = ( isset( $_REQUEST['s'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['s'] ) ) : '' );
 
         $prepare_vars = [];
@@ -125 +125 @@
             $prepare_vars[] = $ptype;
         } else {
-            $args = [ 'public' => true ];
+            $args         = [ 'public' => true ];
             $post_types   = get_post_types( $args );
             $ptype_clause = sprintf(
                 'and a.post_type IN (%s)',
-                implode( ',', array_map( function( $a ) { return "'" . $a . "'"; }, $post_types ) )
+                implode(
+                    ',',
+                    array_map(
+                        function( $a ) {
+                            return "'" . $a . "'";
+                        },
+                        $post_types
+                    )
+                )
             );
         }
@@ -251 +259 @@
             return;
         }
-        $ptype     = isset( $_GET['ptype'] ) ? $_GET['ptype'] : '';
-        $startdate = isset( $_GET['startdate'] ) ? $_GET['startdate'] : '';
-        $enddate   = isset( $_GET['enddate'] ) ? $_GET['enddate'] : '';
+        $ptype     = isset( $_GET['ptype'] ) ? sanitize_text_field( wp_unslash( $_GET['ptype'] ) ) : '';
+        $startdate = isset( $_GET['startdate'] ) ? sanitize_text_field( wp_unslash( $_GET['startdate'] ) ) : '';
+        $enddate   = isset( $_GET['enddate'] ) ? sanitize_text_field( wp_unslash( $_GET['enddate'] ) ) : '';
         ?>
         <div class="alignleft actions">
-            <label class="screen-reader-text" for="ptype"><?php _e( 'Post type&hellip;', 'buua' ) ?></label>
+            <label class="screen-reader-text" for="ptype"><?php esc_html_e( 'Post type&hellip;', 'buua' ); ?></label>
             <select name="ptype" id="ptype">
-                <option <?php selected( $ptype, '' ); ?> value=''><?php _e( 'Post type&hellip;', 'buua' ); ?></option>
+                <option <?php selected( $ptype, '' ); ?> value=''><?php esc_html_e( 'Post type&hellip;', 'buua' ); ?></option>
                 <?php
                 $args       = [ 'public' => true ];
@@ -266 +274 @@
                 <?php endforeach ?>
             </select>
-            <input placeholder="<?php _e( 'Start date', 'buua' ); ?>" type="text" id="startdate" name="startdate" class="startdate datepicker" value="<?php echo esc_html( $startdate ); ?>" />
-            <input placeholder="<?php _e( 'End date', 'buua' ); ?>" type="text" id="enddate" name ="enddate" class="enddate datepicker" value="<?php echo $enddate; ?>" />
+            <input placeholder="<?php esc_attr_e( 'Start date', 'buua' ); ?>" type="text" id="startdate" name="startdate" class="startdate datepicker" value="<?php echo esc_html( $startdate ); ?>" />
+            <input placeholder="<?php esc_attr_e( 'End date', 'buua' ); ?>" type="text" id="enddate" name ="enddate" class="enddate datepicker" value="<?php echo $enddate; ?>" />
             <?php submit_button( __( 'Filter', 'buua' ), 'button', false, false, [ 'id' => 'post-query-submit' ] ); ?>
         </div>

user-activity/trunk/readme.txt

@@ -3 +3 @@
 Donate link: https://www.paypal.me/JonasNordstrom
 Tags: users, list, admin
-Requires PHP: 5.4
+Requires PHP: 5.6
 Requires at least: 4.7.0
-Tested up to: 4.9.8
-Stable tag: 0.9.3
+Tested up to: 5.1.1
+Stable tag: 1.0.0
 
 List number of posts per user. You can limit the search by date, post type and user name.
@@ -12 +12 @@
 == Description ==
 
-Get an overview over how active your users has been during a specified time period. This plugin will list all the users,
+Get an overview of how active your users has been during a specified time period. This plugin will list all the users,
 the number of posts/pages they have created and when their latest post was created.
 
@@ -39 +39 @@
 == Changelog ==
 
+= 1.0.0 =
+Version bump and code cleanup
+
+= 0.9.4 =
+
+* WP version bump
+
 = 0.9.3 =
 

user-activity/trunk/user-activity.php

@@ -4 +4 @@
 Plugin URI: http://jonasnordstrom.se/plugins/
 Description: List number of posts per user. You can limit the search by date, post type and user name.
-Version: 0.9.3
+Version: 1.0.0
 Author: Jonas Nordström
 Author URI: http://jonasnordstrom.se/