@@ -69,7 +69,7 @@ |
||
| 69 | 69 | * @param \GV\Entry $entry The entry. |
| 70 | 70 | * @param \GV\Request $request The request context. |
| 71 | 71 | * |
| 72 | - * @return mixed $value The filtered value. |
|
| 72 | + * @return string $value The filtered value. |
|
| 73 | 73 | */ |
| 74 | 74 | public function get_value( $value, $field, $view, $source, $entry, $request ) { |
| 75 | 75 | return $this->get_content( $value, $entry->as_entry(), $field->as_configuration() ); |
@@ -49,7 +49,7 @@ discard block |
||
| 49 | 49 | public function get_content( $output = '', $entry = array(), $field_settings = array(), $field = array() ) { |
| 50 | 50 | |
| 51 | 51 | /** Overridden by a template. */ |
| 52 | - if( ! empty( $field['field_path'] ) ) { return $output; } |
|
| 52 | + if ( ! empty( $field[ 'field_path' ] ) ) { return $output; } |
|
| 53 | 53 | |
| 54 | 54 | $amount = \GV\Utils::get( $entry, 'payment_amount' ); |
| 55 | 55 | $return = GFCommon::to_money( $amount, \GV\Utils::get( $entry, 'currency' ) ); |
@@ -88,14 +88,14 @@ discard block |
||
| 88 | 88 | * |
| 89 | 89 | * @return string Original text if {date_created} isn't found. Otherwise, replaced text. |
| 90 | 90 | */ |
| 91 | - public function replace_merge_tag( $matches = array(), $text = '', $form = array(), $entry = array(), $url_encode = false, $esc_html = false ) { |
|
| 91 | + public function replace_merge_tag( $matches = array(), $text = '', $form = array(), $entry = array(), $url_encode = false, $esc_html = false ) { |
|
| 92 | 92 | |
| 93 | 93 | $return = $text; |
| 94 | 94 | |
| 95 | 95 | foreach ( $matches as $match ) { |
| 96 | 96 | |
| 97 | - $full_tag = $match[0]; |
|
| 98 | - $modifier = isset( $match[1] ) ? $match[1] : false; |
|
| 97 | + $full_tag = $match[ 0 ]; |
|
| 98 | + $modifier = isset( $match[ 1 ] ) ? $match[ 1 ] : false; |
|
| 99 | 99 | |
| 100 | 100 | $amount = \GV\Utils::get( $entry, 'payment_amount' ); |
| 101 | 101 | |
@@ -38,10 +38,10 @@ discard block |
||
| 38 | 38 | $this->api_data = $_api_data; |
| 39 | 39 | $this->name = plugin_basename( $_plugin_file ); |
| 40 | 40 | $this->slug = basename( $_plugin_file, '.php' ); |
| 41 | - $this->version = $_api_data['version']; |
|
| 42 | - $this->wp_override = isset( $_api_data['wp_override'] ) ? (bool) $_api_data['wp_override'] : false; |
|
| 43 | - $this->beta = ! empty( $this->api_data['beta'] ) ? true : false; |
|
| 44 | - $this->cache_key = md5( serialize( $this->slug . $this->api_data['license'] . $this->beta ) ); |
|
| 41 | + $this->version = $_api_data[ 'version' ]; |
|
| 42 | + $this->wp_override = isset( $_api_data[ 'wp_override' ] ) ? (bool)$_api_data[ 'wp_override' ] : false; |
|
| 43 | + $this->beta = ! empty( $this->api_data[ 'beta' ] ) ? true : false; |
|
| 44 | + $this->cache_key = md5( serialize( $this->slug . $this->api_data[ 'license' ] . $this->beta ) ); |
|
| 45 | 45 | |
| 46 | 46 | $edd_plugin_data[ $this->slug ] = $this->api_data; |
| 47 | 47 | |
@@ -133,11 +133,11 @@ discard block |
||
| 133 | 133 | return; |
| 134 | 134 | } |
| 135 | 135 | |
| 136 | - if( ! current_user_can( 'update_plugins' ) ) { |
|
| 136 | + if ( ! current_user_can( 'update_plugins' ) ) { |
|
| 137 | 137 | return; |
| 138 | 138 | } |
| 139 | 139 | |
| 140 | - if( ! is_multisite() ) { |
|
| 140 | + if ( ! is_multisite() ) { |
|
| 141 | 141 | return; |
| 142 | 142 | } |
| 143 | 143 | |
@@ -212,7 +212,7 @@ discard block |
||
| 212 | 212 | '<a target="_blank" class="thickbox" href="' . esc_url( $changelog_link ) . '">', |
| 213 | 213 | esc_html( $version_info->new_version ), |
| 214 | 214 | '</a>', |
| 215 | - '<a href="' . esc_url( wp_nonce_url( self_admin_url( 'update.php?action=upgrade-plugin&plugin=' ) . $this->name, 'upgrade-plugin_' . $this->name ) ) .'">', |
|
| 215 | + '<a href="' . esc_url( wp_nonce_url( self_admin_url( 'update.php?action=upgrade-plugin&plugin=' ) . $this->name, 'upgrade-plugin_' . $this->name ) ) . '">', |
|
| 216 | 216 | '</a>' |
| 217 | 217 | ); |
| 218 | 218 | } |
@@ -256,7 +256,7 @@ discard block |
||
| 256 | 256 | ) |
| 257 | 257 | ); |
| 258 | 258 | |
| 259 | - $cache_key = 'edd_api_request_' . md5( serialize( $this->slug . $this->api_data['license'] . $this->beta ) ); |
|
| 259 | + $cache_key = 'edd_api_request_' . md5( serialize( $this->slug . $this->api_data[ 'license' ] . $this->beta ) ); |
|
| 260 | 260 | |
| 261 | 261 | // Get the transient where we store the api request for this plugin for 24 hours |
| 262 | 262 | $edd_api_request_transient = $this->get_cached_version_info( $cache_key ); |
@@ -311,7 +311,7 @@ discard block |
||
| 311 | 311 | |
| 312 | 312 | $verify_ssl = $this->verify_ssl(); |
| 313 | 313 | if ( strpos( $url, 'https://' ) !== false && strpos( $url, 'edd_action=package_download' ) ) { |
| 314 | - $args['sslverify'] = $verify_ssl; |
|
| 314 | + $args[ 'sslverify' ] = $verify_ssl; |
|
| 315 | 315 | } |
| 316 | 316 | return $args; |
| 317 | 317 | |
@@ -334,24 +334,24 @@ discard block |
||
| 334 | 334 | |
| 335 | 335 | $data = array_merge( $this->api_data, $_data ); |
| 336 | 336 | |
| 337 | - if ( $data['slug'] != $this->slug ) { |
|
| 337 | + if ( $data[ 'slug' ] != $this->slug ) { |
|
| 338 | 338 | return; |
| 339 | 339 | } |
| 340 | 340 | |
| 341 | - if( $this->api_url == trailingslashit (home_url() ) ) { |
|
| 341 | + if ( $this->api_url == trailingslashit( home_url() ) ) { |
|
| 342 | 342 | return false; // Don't allow a plugin to ping itself |
| 343 | 343 | } |
| 344 | 344 | |
| 345 | 345 | $api_params = array( |
| 346 | 346 | 'edd_action' => 'get_version', |
| 347 | - 'license' => ! empty( $data['license'] ) ? $data['license'] : '', |
|
| 348 | - 'item_name' => isset( $data['item_name'] ) ? $data['item_name'] : false, |
|
| 349 | - 'item_id' => isset( $data['item_id'] ) ? $data['item_id'] : false, |
|
| 350 | - 'version' => isset( $data['version'] ) ? $data['version'] : false, |
|
| 351 | - 'slug' => $data['slug'], |
|
| 352 | - 'author' => $data['author'], |
|
| 347 | + 'license' => ! empty( $data[ 'license' ] ) ? $data[ 'license' ] : '', |
|
| 348 | + 'item_name' => isset( $data[ 'item_name' ] ) ? $data[ 'item_name' ] : false, |
|
| 349 | + 'item_id' => isset( $data[ 'item_id' ] ) ? $data[ 'item_id' ] : false, |
|
| 350 | + 'version' => isset( $data[ 'version' ] ) ? $data[ 'version' ] : false, |
|
| 351 | + 'slug' => $data[ 'slug' ], |
|
| 352 | + 'author' => $data[ 'author' ], |
|
| 353 | 353 | 'url' => home_url(), |
| 354 | - 'beta' => ! empty( $data['beta'] ), |
|
| 354 | + 'beta' => ! empty( $data[ 'beta' ] ), |
|
| 355 | 355 | ); |
| 356 | 356 | |
| 357 | 357 | $verify_ssl = $this->verify_ssl(); |
@@ -371,9 +371,9 @@ discard block |
||
| 371 | 371 | $request->banners = maybe_unserialize( $request->banners ); |
| 372 | 372 | } |
| 373 | 373 | |
| 374 | - if( ! empty( $request->sections ) ) { |
|
| 375 | - foreach( $request->sections as $key => $section ) { |
|
| 376 | - $request->$key = (array) $section; |
|
| 374 | + if ( ! empty( $request->sections ) ) { |
|
| 375 | + foreach ( $request->sections as $key => $section ) { |
|
| 376 | + $request->$key = (array)$section; |
|
| 377 | 377 | } |
| 378 | 378 | } |
| 379 | 379 | |
@@ -384,37 +384,37 @@ discard block |
||
| 384 | 384 | |
| 385 | 385 | global $edd_plugin_data; |
| 386 | 386 | |
| 387 | - if( empty( $_REQUEST['edd_sl_action'] ) || 'view_plugin_changelog' != $_REQUEST['edd_sl_action'] ) { |
|
| 387 | + if ( empty( $_REQUEST[ 'edd_sl_action' ] ) || 'view_plugin_changelog' != $_REQUEST[ 'edd_sl_action' ] ) { |
|
| 388 | 388 | return; |
| 389 | 389 | } |
| 390 | 390 | |
| 391 | - if( empty( $_REQUEST['plugin'] ) ) { |
|
| 391 | + if ( empty( $_REQUEST[ 'plugin' ] ) ) { |
|
| 392 | 392 | return; |
| 393 | 393 | } |
| 394 | 394 | |
| 395 | - if( empty( $_REQUEST['slug'] ) ) { |
|
| 395 | + if ( empty( $_REQUEST[ 'slug' ] ) ) { |
|
| 396 | 396 | return; |
| 397 | 397 | } |
| 398 | 398 | |
| 399 | - if( ! current_user_can( 'update_plugins' ) ) { |
|
| 399 | + if ( ! current_user_can( 'update_plugins' ) ) { |
|
| 400 | 400 | wp_die( __( 'You do not have permission to install plugin updates', 'gravityview' ), __( 'Error', 'gravityview' ), array( 'response' => 403 ) ); |
| 401 | 401 | } |
| 402 | 402 | |
| 403 | - $data = $edd_plugin_data[ $_REQUEST['slug'] ]; |
|
| 404 | - $beta = ! empty( $data['beta'] ) ? true : false; |
|
| 405 | - $cache_key = md5( 'edd_plugin_' . sanitize_key( $_REQUEST['plugin'] ) . '_' . $beta . '_version_info' ); |
|
| 403 | + $data = $edd_plugin_data[ $_REQUEST[ 'slug' ] ]; |
|
| 404 | + $beta = ! empty( $data[ 'beta' ] ) ? true : false; |
|
| 405 | + $cache_key = md5( 'edd_plugin_' . sanitize_key( $_REQUEST[ 'plugin' ] ) . '_' . $beta . '_version_info' ); |
|
| 406 | 406 | $version_info = $this->get_cached_version_info( $cache_key ); |
| 407 | 407 | |
| 408 | - if( false === $version_info ) { |
|
| 408 | + if ( false === $version_info ) { |
|
| 409 | 409 | |
| 410 | 410 | $api_params = array( |
| 411 | 411 | 'edd_action' => 'get_version', |
| 412 | - 'item_name' => isset( $data['item_name'] ) ? $data['item_name'] : false, |
|
| 413 | - 'item_id' => isset( $data['item_id'] ) ? $data['item_id'] : false, |
|
| 414 | - 'slug' => $_REQUEST['slug'], |
|
| 415 | - 'author' => $data['author'], |
|
| 412 | + 'item_name' => isset( $data[ 'item_name' ] ) ? $data[ 'item_name' ] : false, |
|
| 413 | + 'item_id' => isset( $data[ 'item_id' ] ) ? $data[ 'item_id' ] : false, |
|
| 414 | + 'slug' => $_REQUEST[ 'slug' ], |
|
| 415 | + 'author' => $data[ 'author' ], |
|
| 416 | 416 | 'url' => home_url(), |
| 417 | - 'beta' => ! empty( $data['beta'] ) |
|
| 417 | + 'beta' => ! empty( $data[ 'beta' ] ) |
|
| 418 | 418 | ); |
| 419 | 419 | |
| 420 | 420 | $verify_ssl = $this->verify_ssl(); |
@@ -431,9 +431,9 @@ discard block |
||
| 431 | 431 | $version_info = false; |
| 432 | 432 | } |
| 433 | 433 | |
| 434 | - if( ! empty( $version_info ) ) { |
|
| 435 | - foreach( $version_info->sections as $key => $section ) { |
|
| 436 | - $version_info->$key = (array) $section; |
|
| 434 | + if ( ! empty( $version_info ) ) { |
|
| 435 | + foreach ( $version_info->sections as $key => $section ) { |
|
| 436 | + $version_info->$key = (array)$section; |
|
| 437 | 437 | } |
| 438 | 438 | } |
| 439 | 439 | |
@@ -441,8 +441,8 @@ discard block |
||
| 441 | 441 | |
| 442 | 442 | } |
| 443 | 443 | |
| 444 | - if( ! empty( $version_info ) && isset( $version_info->sections['changelog'] ) ) { |
|
| 445 | - echo '<div style="background:#fff;padding:10px;">' . $version_info->sections['changelog'] . '</div>'; |
|
| 444 | + if ( ! empty( $version_info ) && isset( $version_info->sections[ 'changelog' ] ) ) { |
|
| 445 | + echo '<div style="background:#fff;padding:10px;">' . $version_info->sections[ 'changelog' ] . '</div>'; |
|
| 446 | 446 | } |
| 447 | 447 | |
| 448 | 448 | exit; |
@@ -450,23 +450,23 @@ discard block |
||
| 450 | 450 | |
| 451 | 451 | public function get_cached_version_info( $cache_key = '' ) { |
| 452 | 452 | |
| 453 | - if( empty( $cache_key ) ) { |
|
| 453 | + if ( empty( $cache_key ) ) { |
|
| 454 | 454 | $cache_key = $this->cache_key; |
| 455 | 455 | } |
| 456 | 456 | |
| 457 | 457 | $cache = get_option( $cache_key ); |
| 458 | 458 | |
| 459 | - if( empty( $cache['timeout'] ) || current_time( 'timestamp' ) > $cache['timeout'] ) { |
|
| 459 | + if ( empty( $cache[ 'timeout' ] ) || current_time( 'timestamp' ) > $cache[ 'timeout' ] ) { |
|
| 460 | 460 | return false; // Cache is expired |
| 461 | 461 | } |
| 462 | 462 | |
| 463 | - return json_decode( $cache['value'] ); |
|
| 463 | + return json_decode( $cache[ 'value' ] ); |
|
| 464 | 464 | |
| 465 | 465 | } |
| 466 | 466 | |
| 467 | 467 | public function set_version_info_cache( $value = '', $cache_key = '' ) { |
| 468 | 468 | |
| 469 | - if( empty( $cache_key ) ) { |
|
| 469 | + if ( empty( $cache_key ) ) { |
|
| 470 | 470 | $cache_key = $this->cache_key; |
| 471 | 471 | } |
| 472 | 472 | |
@@ -486,7 +486,7 @@ discard block |
||
| 486 | 486 | * @return bool |
| 487 | 487 | */ |
| 488 | 488 | private function verify_ssl() { |
| 489 | - return (bool) apply_filters( 'edd_sl_api_request_verify_ssl', true, $this ); |
|
| 489 | + return (bool)apply_filters( 'edd_sl_api_request_verify_ssl', true, $this ); |
|
| 490 | 490 | } |
| 491 | 491 | |
| 492 | 492 | } |
@@ -2,7 +2,9 @@ |
||
| 2 | 2 | namespace GV; |
| 3 | 3 | |
| 4 | 4 | // Exit if accessed directly |
| 5 | -if ( ! defined( 'ABSPATH' ) ) exit; |
|
| 5 | +if ( ! defined( 'ABSPATH' ) ) { |
|
| 6 | + exit; |
|
| 7 | +} |
|
| 6 | 8 | |
| 7 | 9 | /** |
| 8 | 10 | * Allows plugins to use their own update API. |
@@ -78,7 +78,7 @@ |
||
| 78 | 78 | * @uses api_request() |
| 79 | 79 | * |
| 80 | 80 | * @param array $_transient_data Update array build by WordPress. |
| 81 | - * @return array Modified update array with custom plugin data. |
|
| 81 | + * @return \stdClass Modified update array with custom plugin data. |
|
| 82 | 82 | */ |
| 83 | 83 | public function check_update( $_transient_data ) { |
| 84 | 84 | |
@@ -25,7 +25,7 @@ discard block |
||
| 25 | 25 | printf( '<body style="padding: 0; margin: 0; font-family: -apple-system, BlinkMacSystemFont, \'Segoe UI\', Roboto, Oxygen-Sans, Ubuntu, Cantarell, \'Helvetica Neue\', sans-serif;">' ); |
| 26 | 26 | printf( '<img src="' . plugins_url( 'assets/images/astronaut-200x263.png', GRAVITYVIEW_FILE ) . '" alt="The GravityView Astronaut Says:" style="float: left; height: 60px; margin-right : 10px;" />' ); |
| 27 | 27 | printf( __( "%sGravityView requires PHP Version %s or newer.%s \n\nYou're using Version %s. Please ask your host to upgrade your server's PHP.", 'gravityview' ), |
| 28 | - '<h3 style="font-size:16px; margin: 0 0 8px 0;">', GV_MIN_PHP_VERSION , "</h3>\n\n", $version ); |
|
| 28 | + '<h3 style="font-size:16px; margin: 0 0 8px 0;">', GV_MIN_PHP_VERSION, "</h3>\n\n", $version ); |
|
| 29 | 29 | printf( '</body>' ); |
| 30 | 30 | } |
| 31 | 31 | |
@@ -34,7 +34,7 @@ discard block |
||
| 34 | 34 | } |
| 35 | 35 | |
| 36 | 36 | /** The future branch of GravityView requires PHP 5.3+ namespaces and SPL. */ |
| 37 | -if ( version_compare( phpversion(), '5.3.0' , '<' ) ) { |
|
| 37 | +if ( version_compare( phpversion(), '5.3.0', '<' ) ) { |
|
| 38 | 38 | require GRAVITYVIEW_DIR . 'future/_stubs.php'; |
| 39 | 39 | |
| 40 | 40 | /** All looks fine. */ |
@@ -62,7 +62,6 @@ |
||
| 62 | 62 | * |
| 63 | 63 | * Internal static cache for gets, and whatnot. |
| 64 | 64 | * This is not persistent, resets across requests. |
| 65 | - |
|
| 66 | 65 | * @internal |
| 67 | 66 | */ |
| 68 | 67 | private static $cache = array(); |
@@ -209,7 +209,7 @@ discard block |
||
| 209 | 209 | $slug = apply_filters( 'gravityview_slug', 'view' ); |
| 210 | 210 | $rule = array( sprintf( '%s/([^/]+)/csv/?', $slug ), 'index.php?gravityview=$matches[1]&csv=1', 'top' ); |
| 211 | 211 | |
| 212 | - add_filter( 'query_vars', function( $query_vars ) { |
|
| 212 | + add_filter( 'query_vars', function( $query_vars ) { |
|
| 213 | 213 | $query_vars[] = 'csv'; |
| 214 | 214 | return $query_vars; |
| 215 | 215 | } ); |
@@ -254,8 +254,9 @@ discard block |
||
| 254 | 254 | * Check permissions. |
| 255 | 255 | */ |
| 256 | 256 | while ( $error = $view->can_render( null, $request ) ) { |
| 257 | - if ( ! is_wp_error( $error ) ) |
|
| 258 | - break; |
|
| 257 | + if ( ! is_wp_error( $error ) ) { |
|
| 258 | + break; |
|
| 259 | + } |
|
| 259 | 260 | |
| 260 | 261 | switch ( str_replace( 'gravityview/', '', $error->get_error_code() ) ) { |
| 261 | 262 | case 'post_password_required': |
@@ -795,6 +795,7 @@ |
||
| 795 | 795 | * Retrieve the entries for the current view and request. |
| 796 | 796 | * |
| 797 | 797 | * @param \GV\Request The request. Unused for now. |
| 798 | + * @param Request $request |
|
| 798 | 799 | * |
| 799 | 800 | * @return \GV\Entry_Collection The entries. |
| 800 | 801 | */ |
@@ -108,7 +108,7 @@ discard block |
||
| 108 | 108 | $supports = array( 'title', 'revisions' ); |
| 109 | 109 | |
| 110 | 110 | if ( $is_hierarchical ) { |
| 111 | - $supports[] = 'page-attributes'; |
|
| 111 | + $supports[ ] = 'page-attributes'; |
|
| 112 | 112 | } |
| 113 | 113 | |
| 114 | 114 | /** |
@@ -210,11 +210,11 @@ discard block |
||
| 210 | 210 | $rule = array( sprintf( '%s/([^/]+)/csv/?', $slug ), 'index.php?gravityview=$matches[1]&csv=1', 'top' ); |
| 211 | 211 | |
| 212 | 212 | add_filter( 'query_vars', function( $query_vars ) { |
| 213 | - $query_vars[] = 'csv'; |
|
| 213 | + $query_vars[ ] = 'csv'; |
|
| 214 | 214 | return $query_vars; |
| 215 | 215 | } ); |
| 216 | 216 | |
| 217 | - if ( ! isset( $wp_rewrite->extra_rules_top[ $rule[0] ] ) ) { |
|
| 217 | + if ( ! isset( $wp_rewrite->extra_rules_top[ $rule[ 0 ] ] ) ) { |
|
| 218 | 218 | call_user_func_array( 'add_rewrite_rule', $rule ); |
| 219 | 219 | } |
| 220 | 220 | } |
@@ -283,13 +283,13 @@ discard block |
||
| 283 | 283 | return $content; |
| 284 | 284 | } |
| 285 | 285 | |
| 286 | - $is_admin_and_can_view = $view->settings->get( 'admin_show_all_statuses' ) && \GVCommon::has_cap('gravityview_moderate_entries', $view->ID ); |
|
| 286 | + $is_admin_and_can_view = $view->settings->get( 'admin_show_all_statuses' ) && \GVCommon::has_cap( 'gravityview_moderate_entries', $view->ID ); |
|
| 287 | 287 | |
| 288 | 288 | /** |
| 289 | 289 | * Editing a single entry. |
| 290 | 290 | */ |
| 291 | 291 | if ( $entry = $request->is_edit_entry( $view->form ? $view->form->ID : 0 ) ) { |
| 292 | - if ( $entry['status'] != 'active' ) { |
|
| 292 | + if ( $entry[ 'status' ] != 'active' ) { |
|
| 293 | 293 | gravityview()->log->notice( 'Entry ID #{entry_id} is not active', array( 'entry_id' => $entry->ID ) ); |
| 294 | 294 | return __( 'You are not allowed to view this content.', 'gravityview' ); |
| 295 | 295 | } |
@@ -300,7 +300,7 @@ discard block |
||
| 300 | 300 | } |
| 301 | 301 | |
| 302 | 302 | if ( $view->settings->get( 'show_only_approved' ) && ! $is_admin_and_can_view ) { |
| 303 | - if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) ) ) { |
|
| 303 | + if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) ) ) { |
|
| 304 | 304 | gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing', array( 'entry_id' => $entry->ID ) ); |
| 305 | 305 | return __( 'You are not allowed to view this content.', 'gravityview' ); |
| 306 | 306 | } |
@@ -313,7 +313,7 @@ discard block |
||
| 313 | 313 | * Viewing a single entry. |
| 314 | 314 | */ |
| 315 | 315 | } else if ( $entry = $request->is_entry( $view->form ? $view->form->ID : 0 ) ) { |
| 316 | - if ( $entry['status'] != 'active' ) { |
|
| 316 | + if ( $entry[ 'status' ] != 'active' ) { |
|
| 317 | 317 | gravityview()->log->notice( 'Entry ID #{entry_id} is not active', array( 'entry_id' => $entry->ID ) ); |
| 318 | 318 | return __( 'You are not allowed to view this content.', 'gravityview' ); |
| 319 | 319 | } |
@@ -324,7 +324,7 @@ discard block |
||
| 324 | 324 | } |
| 325 | 325 | |
| 326 | 326 | if ( $view->settings->get( 'show_only_approved' ) && ! $is_admin_and_can_view ) { |
| 327 | - if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) ) ) { |
|
| 327 | + if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) ) ) { |
|
| 328 | 328 | gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing', array( 'entry_id' => $entry->ID ) ); |
| 329 | 329 | return __( 'You are not allowed to view this content.', 'gravityview' ); |
| 330 | 330 | } |
@@ -332,7 +332,7 @@ discard block |
||
| 332 | 332 | |
| 333 | 333 | $error = \GVCommon::check_entry_display( $entry->as_entry(), $view ); |
| 334 | 334 | |
| 335 | - if( is_wp_error( $error ) ) { |
|
| 335 | + if ( is_wp_error( $error ) ) { |
|
| 336 | 336 | gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing: {message}', array( 'entry_id' => $entry->ID, 'message' => $error->get_error_message() ) ); |
| 337 | 337 | return __( 'You are not allowed to view this content.', 'gravityview' ); |
| 338 | 338 | } |
@@ -498,7 +498,7 @@ discard block |
||
| 498 | 498 | $join_column = is_numeric( $join_column ) ? GF_Field::by_id( $join, $join_column ) : Internal_Field( $join_column ); |
| 499 | 499 | $join_on_column = is_numeric( $join_on_column ) ? GF_Field::by_id( $join_on, $join_on_column ) : Internal_Field( $join_on_column ); |
| 500 | 500 | |
| 501 | - $joins [] = new Join( $join, $join_column, $join_on, $join_on_column ); |
|
| 501 | + $joins [ ] = new Join( $join, $join_column, $join_on, $join_on_column ); |
|
| 502 | 502 | } |
| 503 | 503 | |
| 504 | 504 | return $joins; |
@@ -540,10 +540,10 @@ discard block |
||
| 540 | 540 | |
| 541 | 541 | list( $join, $join_column, $join_on, $join_on_column ) = $meta; |
| 542 | 542 | |
| 543 | - $forms_ids [] = GF_Form::by_id( $join_on ); |
|
| 543 | + $forms_ids [ ] = GF_Form::by_id( $join_on ); |
|
| 544 | 544 | } |
| 545 | 545 | |
| 546 | - return ( !empty( $forms_ids) ) ? $forms_ids : null; |
|
| 546 | + return ( ! empty( $forms_ids ) ) ? $forms_ids : null; |
|
| 547 | 547 | } |
| 548 | 548 | |
| 549 | 549 | /** |
@@ -581,7 +581,7 @@ discard block |
||
| 581 | 581 | if ( ! $view->form ) { |
| 582 | 582 | gravityview()->log->error( 'View #{view_id} tried attaching non-existent Form #{form_id} to it.', array( |
| 583 | 583 | 'view_id' => $view->ID, |
| 584 | - 'form_id' => $view->_gravityview_form_id ? : 0, |
|
| 584 | + 'form_id' => $view->_gravityview_form_id ?: 0, |
|
| 585 | 585 | ) ); |
| 586 | 586 | } |
| 587 | 587 | |
@@ -814,34 +814,33 @@ discard block |
||
| 814 | 814 | * @todo: Stop using _frontend and use something like $request->get_search_criteria() instead |
| 815 | 815 | */ |
| 816 | 816 | $parameters = \GravityView_frontend::get_view_entries_parameters( $this->settings->as_atts(), $this->form->ID ); |
| 817 | - $parameters['context_view_id'] = $this->ID; |
|
| 817 | + $parameters[ 'context_view_id' ] = $this->ID; |
|
| 818 | 818 | $parameters = \GVCommon::calculate_get_entries_criteria( $parameters, $this->form->ID ); |
| 819 | 819 | |
| 820 | 820 | if ( $request instanceof REST\Request ) { |
| 821 | 821 | $atts = $this->settings->as_atts(); |
| 822 | 822 | $paging_parameters = wp_parse_args( $request->get_paging(), array( |
| 823 | - 'paging' => array( 'page_size' => $atts['page_size'] ), |
|
| 823 | + 'paging' => array( 'page_size' => $atts[ 'page_size' ] ), |
|
| 824 | 824 | ) ); |
| 825 | - $parameters['paging'] = $paging_parameters['paging']; |
|
| 825 | + $parameters[ 'paging' ] = $paging_parameters[ 'paging' ]; |
|
| 826 | 826 | } |
| 827 | 827 | |
| 828 | - $page = Utils::get( $parameters['paging'], 'current_page' ) ? |
|
| 829 | - : ( ( ( $parameters['paging']['offset'] - $this->settings->get( 'offset' ) ) / $parameters['paging']['page_size'] ) + 1 ); |
|
| 828 | + $page = Utils::get( $parameters[ 'paging' ], 'current_page' ) ?: ( ( ( $parameters[ 'paging' ][ 'offset' ] - $this->settings->get( 'offset' ) ) / $parameters[ 'paging' ][ 'page_size' ] ) + 1 ); |
|
| 830 | 829 | |
| 831 | 830 | /** |
| 832 | 831 | * Cleanup duplicate field_filter parameters to simplify the query. |
| 833 | 832 | */ |
| 834 | 833 | $unique_field_filters = array(); |
| 835 | - foreach ( $parameters['search_criteria']['field_filters'] as $key => $filter ) { |
|
| 834 | + foreach ( $parameters[ 'search_criteria' ][ 'field_filters' ] as $key => $filter ) { |
|
| 836 | 835 | if ( 'mode' === $key ) { |
| 837 | - $unique_field_filters['mode'] = $filter; |
|
| 836 | + $unique_field_filters[ 'mode' ] = $filter; |
|
| 838 | 837 | } else if ( ! in_array( $filter, $unique_field_filters ) ) { |
| 839 | - $unique_field_filters[] = $filter; |
|
| 838 | + $unique_field_filters[ ] = $filter; |
|
| 840 | 839 | } |
| 841 | 840 | } |
| 842 | - $parameters['search_criteria']['field_filters'] = $unique_field_filters; |
|
| 841 | + $parameters[ 'search_criteria' ][ 'field_filters' ] = $unique_field_filters; |
|
| 843 | 842 | |
| 844 | - if ( ! empty( $parameters['search_criteria']['field_filters'] ) ) { |
|
| 843 | + if ( ! empty( $parameters[ 'search_criteria' ][ 'field_filters' ] ) ) { |
|
| 845 | 844 | gravityview()->log->notice( 'search_criteria/field_filters is not empty, third-party code may be using legacy search_criteria filters.' ); |
| 846 | 845 | } |
| 847 | 846 | |
@@ -849,10 +848,10 @@ discard block |
||
| 849 | 848 | /** |
| 850 | 849 | * New \GF_Query stuff :) |
| 851 | 850 | */ |
| 852 | - $query = new \GF_Query( $this->form->ID, $parameters['search_criteria'], $parameters['sorting'] ); |
|
| 851 | + $query = new \GF_Query( $this->form->ID, $parameters[ 'search_criteria' ], $parameters[ 'sorting' ] ); |
|
| 853 | 852 | |
| 854 | - $query->limit( $parameters['paging']['page_size'] ) |
|
| 855 | - ->offset( ( ( $page - 1 ) * $parameters['paging']['page_size'] ) + $this->settings->get( 'offset' ) ); |
|
| 853 | + $query->limit( $parameters[ 'paging' ][ 'page_size' ] ) |
|
| 854 | + ->offset( ( ( $page - 1 ) * $parameters[ 'paging' ][ 'page_size' ] ) + $this->settings->get( 'offset' ) ); |
|
| 856 | 855 | |
| 857 | 856 | /** |
| 858 | 857 | * Any joins? |
@@ -892,15 +891,15 @@ discard block |
||
| 892 | 891 | } ); |
| 893 | 892 | } else { |
| 894 | 893 | $entries = $this->form->entries |
| 895 | - ->filter( \GV\GF_Entry_Filter::from_search_criteria( $parameters['search_criteria'] ) ) |
|
| 894 | + ->filter( \GV\GF_Entry_Filter::from_search_criteria( $parameters[ 'search_criteria' ] ) ) |
|
| 896 | 895 | ->offset( $this->settings->get( 'offset' ) ) |
| 897 | - ->limit( $parameters['paging']['page_size'] ) |
|
| 896 | + ->limit( $parameters[ 'paging' ][ 'page_size' ] ) |
|
| 898 | 897 | ->page( $page ); |
| 899 | 898 | |
| 900 | - if ( ! empty( $parameters['sorting'] ) && ! empty( $parameters['sorting']['key'] ) ) { |
|
| 899 | + if ( ! empty( $parameters[ 'sorting' ] ) && ! empty( $parameters[ 'sorting' ][ 'key' ] ) ) { |
|
| 901 | 900 | $field = new \GV\Field(); |
| 902 | - $field->ID = $parameters['sorting']['key']; |
|
| 903 | - $direction = strtolower( $parameters['sorting']['direction'] ) == 'asc' ? \GV\Entry_Sort::ASC : \GV\Entry_Sort::DESC; |
|
| 901 | + $field->ID = $parameters[ 'sorting' ][ 'key' ]; |
|
| 902 | + $direction = strtolower( $parameters[ 'sorting' ][ 'direction' ] ) == 'asc' ? \GV\Entry_Sort::ASC : \GV\Entry_Sort::DESC; |
|
| 904 | 903 | $entries = $entries->sort( new \GV\Entry_Sort( $field, $direction ) ); |
| 905 | 904 | } |
| 906 | 905 | } |
@@ -34,7 +34,7 @@ |
||
| 34 | 34 | global $post; |
| 35 | 35 | |
| 36 | 36 | if ( $post ) { |
| 37 | - $context['post'] = $post; |
|
| 37 | + $context[ 'post' ] = $post; |
|
| 38 | 38 | } |
| 39 | 39 | |
| 40 | 40 | \GV\Mocks\Legacy_Context::push( $context ); |
@@ -113,14 +113,14 @@ |
||
| 113 | 113 | |
| 114 | 114 | /** Require critical legacy core files. @todo Deprecate */ |
| 115 | 115 | require_once $this->plugin->dir( 'includes/helper-functions.php' ); |
| 116 | - require_once $this->plugin->dir( 'includes/class-common.php'); |
|
| 117 | - require_once $this->plugin->dir( 'includes/connector-functions.php'); |
|
| 116 | + require_once $this->plugin->dir( 'includes/class-common.php' ); |
|
| 117 | + require_once $this->plugin->dir( 'includes/connector-functions.php' ); |
|
| 118 | 118 | require_once $this->plugin->dir( 'includes/class-gravityview-compatibility.php' ); |
| 119 | 119 | require_once $this->plugin->dir( 'includes/class-gravityview-roles-capabilities.php' ); |
| 120 | 120 | require_once $this->plugin->dir( 'includes/class-gravityview-admin-notices.php' ); |
| 121 | 121 | require_once $this->plugin->dir( 'includes/class-admin.php' ); |
| 122 | - require_once $this->plugin->dir( 'includes/class-post-types.php'); |
|
| 123 | - require_once $this->plugin->dir( 'includes/class-cache.php'); |
|
| 122 | + require_once $this->plugin->dir( 'includes/class-post-types.php' ); |
|
| 123 | + require_once $this->plugin->dir( 'includes/class-cache.php' ); |
|
| 124 | 124 | |
| 125 | 125 | /** |
| 126 | 126 | * GravityView extensions and widgets. |
@@ -78,7 +78,7 @@ |
||
| 78 | 78 | if ( $this->field ) { |
| 79 | 79 | return array( |
| 80 | 80 | 'key' => $this->field->ID, |
| 81 | - 'direction' => $this->direction ? : self::ASC, |
|
| 81 | + 'direction' => $this->direction ?: self::ASC, |
|
| 82 | 82 | 'is_numeric' => self::ALPHA ? true : false, |
| 83 | 83 | ); |
| 84 | 84 | } |
@@ -44,7 +44,7 @@ discard block |
||
| 44 | 44 | if ( ! $entry instanceof Entry ) { |
| 45 | 45 | continue; |
| 46 | 46 | } |
| 47 | - $_entry->entries[ $entry['form_id'] ] = &$entry; |
|
| 47 | + $_entry->entries[ $entry[ 'form_id' ] ] = &$entry; |
|
| 48 | 48 | } |
| 49 | 49 | return $_entry; |
| 50 | 50 | } |
@@ -65,7 +65,7 @@ discard block |
||
| 65 | 65 | |
| 66 | 66 | foreach ( $this->entries as $entry ) { |
| 67 | 67 | $entry = $entry->as_entry(); |
| 68 | - $_entry['_multi'][ $entry['form_id'] ] = $entry; |
|
| 68 | + $_entry[ '_multi' ][ $entry[ 'form_id' ] ] = $entry; |
|
| 69 | 69 | } |
| 70 | 70 | } |
| 71 | 71 | |
@@ -50,8 +50,8 @@ |
||
| 50 | 50 | /** |
| 51 | 51 | * By array. |
| 52 | 52 | */ |
| 53 | - if ( is_array( $view ) && ! empty( $view['id'] ) ) { |
|
| 54 | - return $this->get( $view['id'] ); |
|
| 53 | + if ( is_array( $view ) && ! empty( $view[ 'id' ] ) ) { |
|
| 54 | + return $this->get( $view[ 'id' ] ); |
|
| 55 | 55 | } |
| 56 | 56 | |
| 57 | 57 | /** |