Completed
Push — master ( 8e42dd...718d7d )
by Jamie
03:46
created
classes/helpers/FrmEntriesListHelper.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 			$join_form_in_query = true;
24 24
 		}
25 25
 
26
-		$s = isset( $_REQUEST['s'] ) ? stripslashes($_REQUEST['s']) : '';
26
+		$s = isset( $_REQUEST['s'] ) ? stripslashes( $_REQUEST['s'] ) : '';
27 27
 
28 28
 	    if ( $s != '' && FrmAppHelper::pro_is_installed() ) {
29 29
 	        $fid = isset( $_REQUEST['fid'] ) ? sanitize_title( $_REQUEST['fid'] ) : '';
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 	    }
32 32
 
33 33
         $orderby = isset( $_REQUEST['orderby'] ) ? sanitize_title( $_REQUEST['orderby'] ) : $default_orderby;
34
-        if ( strpos($orderby, 'meta') !== false ) {
34
+        if ( strpos( $orderby, 'meta' ) !== false ) {
35 35
             $order_field_type = FrmField::get_type( str_replace( 'meta_', '', $orderby ) );
36 36
 			$orderby .= in_array( $order_field_type, array( 'number', 'scale' ) ) ? ' +0 ' : '';
37 37
         }
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
 		$start = (int) isset( $_REQUEST['start'] ) ? absint( $_REQUEST['start'] ) : ( ( $page - 1 ) * $per_page );
44 44
 
45 45
 		$this->items = FrmEntry::getAll( $s_query, $order, ' LIMIT ' . $start . ',' . $per_page, true, $join_form_in_query );
46
-        $total_items = FrmEntry::getRecordCount($s_query);
46
+        $total_items = FrmEntry::getRecordCount( $s_query );
47 47
 
48 48
 		$this->set_pagination_args( array(
49 49
 			'total_items' => $total_items,
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
 
54 54
 	public function no_items() {
55 55
         $s = isset( $_REQUEST['s'] ) ? $_REQUEST['s'] : '';
56
-	    if ( ! empty($s) ) {
56
+	    if ( ! empty( $s ) ) {
57 57
             _e( 'No Entries Found', 'formidable' );
58 58
             return;
59 59
         }
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 		$form = $this->params['form'];
63 63
 
64 64
         if ( $form_id ) {
65
-            $form = FrmForm::getOne($form_id);
65
+            $form = FrmForm::getOne( $form_id );
66 66
         }
67 67
         $colspan = $this->get_column_count();
68 68
 
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 
121 121
 		$r = "<tr id='item-action-{$item->id}'$style>";
122 122
 
123
-		list( $columns, $hidden, , $primary ) = $this->get_column_info();
123
+		list( $columns, $hidden,, $primary ) = $this->get_column_info();
124 124
         $action_col = false;
125 125
 
126 126
 		foreach ( $columns as $column_name => $column_display_name ) {
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
             }
138 138
 
139 139
 			$attributes = 'class="' . esc_attr( $class ) . '"';
140
-			unset($class);
140
+			unset( $class );
141 141
 			$attributes .= ' data-colname="' . $column_display_name . '"';
142 142
 
143 143
 			$form_id = $this->params['form'] ? $this->params['form'] : 0;
@@ -156,24 +156,24 @@  discard block
 block discarded – undo
156 156
 				    break;
157 157
 				case 'name':
158 158
 				case 'description':
159
-				    $val = FrmAppHelper::truncate(strip_tags($item->{$col_name}), 100);
159
+				    $val = FrmAppHelper::truncate( strip_tags( $item->{$col_name}), 100 );
160 160
 				    break;
161 161
 				case 'created_at':
162 162
 				case 'updated_at':
163
-				    $date = FrmAppHelper::get_formatted_time($item->{$col_name});
163
+				    $date = FrmAppHelper::get_formatted_time( $item->{$col_name});
164 164
 					$val = '<abbr title="' . esc_attr( FrmAppHelper::get_formatted_time( $item->{$col_name}, '', 'g:i:s A' ) ) . '">' . $date . '</abbr>';
165 165
 					break;
166 166
 				case 'is_draft':
167
-				    $val = empty($item->is_draft) ? __( 'No') : __( 'Yes');
167
+				    $val = empty( $item->is_draft ) ? __( 'No' ) : __( 'Yes' );
168 168
 			        break;
169 169
 				case 'form_id':
170
-				    $val = FrmFormsHelper::edit_form_link($item->form_id);
170
+				    $val = FrmFormsHelper::edit_form_link( $item->form_id );
171 171
     				break;
172 172
 				case 'post_id':
173
-				    $val = FrmAppHelper::post_edit_link($item->post_id);
173
+				    $val = FrmAppHelper::post_edit_link( $item->post_id );
174 174
 				    break;
175 175
 				case 'user_id':
176
-				    $user = get_userdata($item->user_id);
176
+				    $user = get_userdata( $item->user_id );
177 177
 				    $val = $user ? $user->user_login : '';
178 178
 				    break;
179 179
 				case 'parent_item_id':
@@ -198,7 +198,7 @@  discard block
 block discarded – undo
198 198
 			    }
199 199
 			    $r .= '</td>';
200 200
 			}
201
-			unset($val);
201
+			unset( $val );
202 202
 		}
203 203
 		$r .= '</tr>';
204 204
 
@@ -211,12 +211,12 @@  discard block
 block discarded – undo
211 211
     private function get_actions( &$actions, $item, $view_link ) {
212 212
 		$actions['view'] = '<a href="' . esc_url( $view_link ) . '">' . __( 'View', 'formidable' ) . '</a>';
213 213
 
214
-        if ( current_user_can('frm_delete_entries') ) {
214
+        if ( current_user_can( 'frm_delete_entries' ) ) {
215 215
 			$delete_link = '?page=formidable-entries&frm_action=destroy&id=' . $item->id . '&form=' . $this->params['form'];
216 216
 			$actions['delete'] = '<a href="' . esc_url( wp_nonce_url( $delete_link ) ) . '" class="submitdelete" onclick="return confirm(\'' . esc_attr( __( 'Are you sure you want to delete that?', 'formidable' ) ) . '\')">' . __( 'Delete' ) . '</a>';
217 217
 	    }
218 218
 
219
-        $actions = apply_filters('frm_row_actions', $actions, $item);
219
+        $actions = apply_filters( 'frm_row_actions', $actions, $item );
220 220
     }
221 221
 
222 222
 	private function get_column_value( $item, &$val ) {
Please login to merge, or discard this patch.