| @@ -34,10 +34,9 @@ | ||
| 34 | 34 | if ( ! $old_db_version ) { |
| 35 | 35 | $old_db_version = get_option( $this->db_opt_name ); |
| 36 | 36 | } |
| 37 | 37 | |
| 38 | - // phpcs:ignore Universal.Operators.StrictComparisons | |
| 39 | - if ( $this->db_version == $old_db_version ) { | |
| 38 | + if ( $this->db_version === (int) $old_db_version ) { | |
| 40 | 39 | return; |
| 41 | 40 | } |
| 42 | 41 | |
| 43 | 42 | global $wpdb; |
| @@ -136,9 +135,9 @@ | ||
| 136 | 135 | * @param int|string $id |
| 137 | 136 | * |
| 138 | 137 | * @return bool|int |
| 139 | 138 | */ |
| 140 | - public function &destroy( $id ) { | |
| 139 | + public function destroy( $id ) { | |
| 141 | 140 | FrmAppHelper::permission_check( 'administrator' ); |
| 142 | 141 | |
| 143 | 142 | global $wpdb; |
| 144 | 143 | $id = absint( $id ); |
| @@ -148,17 +147,10 @@ | ||
| 148 | 147 | */ |
| 149 | 148 | do_action( 'frm_before_destroy_' . $this->singular, $id ); |
| 150 | 149 | |
| 151 | 150 | // @codingStandardsIgnoreStart |
| 152 | - $result = $wpdb->query( | |
| 153 | - $wpdb->prepare( | |
| 154 | - 'DELETE FROM ' . $wpdb->prefix . $this->table_name . ' WHERE id=%d', | |
| 155 | - $id | |
| 156 | - ) | |
| 157 | - ); | |
| 151 | + return $wpdb->query( $wpdb->prepare( 'DELETE FROM ' . $wpdb->prefix . $this->table_name . ' WHERE id=%d', $id ) ); | |
| 158 | 152 | // @codingStandardsIgnoreEnd |
| 159 | - | |
| 160 | - return $result; | |
| 161 | 153 | } |
| 162 | 154 | |
| 163 | 155 | /** |
| 164 | 156 | * @param int|string $id |
| @@ -166,8 +158,9 @@ | ||
| 166 | 158 | * @return array|object|null |
| 167 | 159 | */ |
| 168 | 160 | public function get_one( $id ) { |
| 169 | 161 | global $wpdb; |
| 162 | + | |
| 170 | 163 | // @codingStandardsIgnoreStart |
| 171 | 164 | return $wpdb->get_row( |
| 172 | 165 | $wpdb->prepare( |
| 173 | 166 | 'SELECT * FROM ' . $wpdb->prefix . $this->table_name . ' WHERE id=%d', |
| @@ -292,11 +285,9 @@ | ||
| 292 | 285 | * |
| 293 | 286 | * @return void |
| 294 | 287 | */ |
| 295 | 288 | private function fill_values( $values, &$new_values ) { |
| 296 | - $defaults = $this->get_defaults(); | |
| 297 | - | |
| 298 | - foreach ( $defaults as $val => $default ) { | |
| 289 | + foreach ( $this->get_defaults() as $val => $default ) { | |
| 299 | 290 | if ( isset( $values[ $val ] ) ) { |
| 300 | 291 | if ( $default['sanitize'] === 'float' ) { |
| 301 | 292 | $new_values[ $val ] = (float) $values[ $val ]; |
| 302 | 293 | } elseif ( ! empty( $default['sanitize'] ) ) { |
| @@ -318,12 +309,14 @@ | ||
| 318 | 309 | private function migrate_data( $old_db_version ) { |
| 319 | 310 | $migrations = array( 4 ); |
| 320 | 311 | |
| 321 | 312 | foreach ( $migrations as $migration ) { |
| 322 | - if ( $this->db_version >= $migration && $old_db_version < $migration ) { | |
| 323 | - $function_name = 'migrate_to_' . $migration; | |
| 324 | - $this->$function_name(); | |
| 313 | + if ( $this->db_version < $migration || $old_db_version >= $migration ) { | |
| 314 | + continue; | |
| 325 | 315 | } |
| 316 | + | |
| 317 | + $function_name = 'migrate_to_' . $migration; | |
| 318 | + $this->$function_name(); | |
| 326 | 319 | } |
| 327 | 320 | } |
| 328 | 321 | |
| 329 | 322 | /** |
