| @@ -9,16 +9,10 @@ | ||
| 9 | 9 | class FrmInbox extends FrmFormApi { |
| 10 | 10 | |
| 11 | 11 | protected $cache_key; |
| 12 | 12 | |
| 13 | - /** | |
| 14 | - * @var string | |
| 15 | - */ | |
| 16 | 13 | private $option = 'frm_inbox'; |
| 17 | 14 | |
| 18 | - /** | |
| 19 | - * @var array|false | |
| 20 | - */ | |
| 21 | 15 | private static $messages = false; |
| 22 | 16 | |
| 23 | 17 | /** |
| 24 | 18 | * @var array |
| @@ -24,9 +18,9 @@ | ||
| 24 | 18 | * @var array |
| 25 | 19 | */ |
| 26 | 20 | private static $banner_messages; |
| 27 | 21 | |
| 28 | - public function __construct() { | |
| 22 | + public function __construct( $for_parent = null ) { | |
| 29 | 23 | $this->set_cache_key(); |
| 30 | 24 | |
| 31 | 25 | if ( false === self::$messages ) { |
| 32 | 26 | $this->set_messages(); |
| @@ -53,19 +47,15 @@ | ||
| 53 | 47 | |
| 54 | 48 | /** |
| 55 | 49 | * @since 4.05 |
| 56 | 50 | * |
| 57 | - * @param false|string $filter | |
| 58 | - * | |
| 59 | - * @return array | |
| 51 | + * @param array|false $filter | |
| 60 | 52 | */ |
| 61 | 53 | public function get_messages( $filter = false ) { |
| 62 | 54 | $messages = self::$messages; |
| 63 | - | |
| 64 | 55 | if ( $filter === 'filter' ) { |
| 65 | 56 | $this->filter_messages( $messages ); |
| 66 | 57 | } |
| 67 | - | |
| 68 | 58 | return $messages; |
| 69 | 59 | } |
| 70 | 60 | |
| 71 | 61 | /** |
| @@ -74,9 +64,8 @@ | ||
| 74 | 64 | * @return void |
| 75 | 65 | */ |
| 76 | 66 | public function set_messages() { |
| 77 | 67 | self::$messages = get_option( $this->option ); |
| 78 | - | |
| 79 | 68 | if ( ! is_array( self::$messages ) ) { |
| 80 | 69 | self::$messages = array(); |
| 81 | 70 | } |
| 82 | 71 | |
| @@ -94,10 +83,9 @@ | ||
| 94 | 83 | * @return void |
| 95 | 84 | */ |
| 96 | 85 | private function add_api_messages() { |
| 97 | 86 | $api = $this->get_api_info(); |
| 98 | - | |
| 99 | - if ( ! $api ) { | |
| 87 | + if ( empty( $api ) ) { | |
| 100 | 88 | return; |
| 101 | 89 | } |
| 102 | 90 | |
| 103 | 91 | foreach ( $api as $message ) { |
| @@ -111,10 +99,10 @@ | ||
| 111 | 99 | * @return void |
| 112 | 100 | */ |
| 113 | 101 | public function add_message( $message ) { |
| 114 | 102 | if ( ! is_array( $message ) || ! isset( $message['key'] ) ) { |
| 115 | - // If the API response is invalid, $message may not be an array. | |
| 116 | - // If there are no messages from the API, it is returning a "No Entries Found" item with no key, so check for a key as well. | |
| 103 | + // if the API response is invalid, $message may not be an array. | |
| 104 | + // if there are no messages from the API, it is returning a "No Entries Found" item with no key, so check for a key as well. | |
| 117 | 105 | return; |
| 118 | 106 | } |
| 119 | 107 | |
| 120 | 108 | if ( isset( self::$messages[ $message['key'] ] ) && ! isset( $message['force'] ) ) { |
| @@ -121,9 +109,9 @@ | ||
| 121 | 109 | // Don't replace messages unless required. |
| 122 | 110 | return; |
| 123 | 111 | } |
| 124 | 112 | |
| 125 | - if ( ! $this->within_valid_timeframe( $message ) ) { | |
| 113 | + if ( $this->is_expired( $message ) ) { | |
| 126 | 114 | return; |
| 127 | 115 | } |
| 128 | 116 | |
| 129 | 117 | if ( isset( self::$messages[ $message['key'] ] ) ) { |
| @@ -167,19 +155,16 @@ | ||
| 167 | 155 | * @return void |
| 168 | 156 | */ |
| 169 | 157 | private function clean_messages() { |
| 170 | 158 | $removed = false; |
| 171 | - | |
| 172 | 159 | foreach ( self::$messages as $t => $message ) { |
| 173 | 160 | $read = ! empty( $message['read'] ) && isset( $message['read'][ get_current_user_id() ] ) && $message['read'][ get_current_user_id() ] < strtotime( '-1 month' ); |
| 174 | - $dismissed = ! empty( $message['dismissed'] ) && isset( $message['dismissed'][ get_current_user_id() ] ) && $message['dismissed'][ get_current_user_id() ] < strtotime( '-1 week' ); // phpcs:ignore SlevomatCodingStandard.Files.LineLength.LineTooLong | |
| 175 | - | |
| 176 | - if ( ! $read && ! $dismissed && $this->within_valid_timeframe( $message ) ) { | |
| 177 | - continue; | |
| 161 | + $dismissed = ! empty( $message['dismissed'] ) && isset( $message['dismissed'][ get_current_user_id() ] ) && $message['dismissed'][ get_current_user_id() ] < strtotime( '-1 week' ); | |
| 162 | + $expired = $this->is_expired( $message ); | |
| 163 | + if ( $read || $expired || $dismissed ) { | |
| 164 | + unset( self::$messages[ $t ] ); | |
| 165 | + $removed = true; | |
| 178 | 166 | } |
| 179 | - | |
| 180 | - unset( self::$messages[ $t ] ); | |
| 181 | - $removed = true; | |
| 182 | 167 | } |
| 183 | 168 | |
| 184 | 169 | if ( $removed ) { |
| 185 | 170 | $this->update_list(); |
| @@ -188,58 +173,28 @@ | ||
| 188 | 173 | |
| 189 | 174 | /** |
| 190 | 175 | * @param array $messages |
| 191 | 176 | * @param string $type |
| 192 | - * | |
| 193 | 177 | * @return void |
| 194 | 178 | */ |
| 195 | 179 | public function filter_messages( &$messages, $type = 'unread' ) { |
| 196 | 180 | $user_id = get_current_user_id(); |
| 197 | - | |
| 198 | 181 | foreach ( $messages as $k => $message ) { |
| 199 | 182 | $dismissed = isset( $message['dismissed'] ) && isset( $message['dismissed'][ $user_id ] ); |
| 200 | - | |
| 201 | - if ( ! $k || ! $this->within_valid_timeframe( $message ) || ( $type === 'dismissed' ) !== $dismissed ) { | |
| 183 | + if ( empty( $k ) || $this->is_expired( $message ) || ( $type === 'dismissed' ) !== $dismissed ) { | |
| 202 | 184 | unset( $messages[ $k ] ); |
| 203 | 185 | } elseif ( ! $this->is_for_user( $message ) ) { |
| 204 | 186 | unset( $messages[ $k ] ); |
| 205 | 187 | } |
| 206 | 188 | } |
| 207 | - | |
| 208 | 189 | $messages = apply_filters( 'frm_filter_inbox', $messages ); |
| 209 | 190 | } |
| 210 | 191 | |
| 211 | 192 | /** |
| 212 | - * Check if a message has started and is not expired. | |
| 213 | - * | |
| 214 | - * @since 6.25 | |
| 215 | - * | |
| 216 | 193 | * @param array $message |
| 217 | 194 | * |
| 218 | 195 | * @return bool |
| 219 | 196 | */ |
| 220 | - private function within_valid_timeframe( $message ) { | |
| 221 | - return $this->has_started( $message ) && ! $this->is_expired( $message ); | |
| 222 | - } | |
| 223 | - | |
| 224 | - /** | |
| 225 | - * Check if a message has actually started, so we can prevent showing something that is queued up early. | |
| 226 | - * | |
| 227 | - * @since 6.25 | |
| 228 | - * | |
| 229 | - * @param array $message | |
| 230 | - * | |
| 231 | - * @return bool | |
| 232 | - */ | |
| 233 | - private function has_started( $message ) { | |
| 234 | - return ! empty( $message['starts'] ) ? $message['starts'] <= time() : true; | |
| 235 | - } | |
| 236 | - | |
| 237 | - /** | |
| 238 | - * @param array $message | |
| 239 | - * | |
| 240 | - * @return bool | |
| 241 | - */ | |
| 242 | 197 | private function is_expired( $message ) { |
| 243 | 198 | return ! empty( $message['expires'] ) && $message['expires'] < time(); |
| 244 | 199 | } |
| 245 | 200 | |
| @@ -246,18 +201,24 @@ | ||
| 246 | 201 | /** |
| 247 | 202 | * Show different messages for different accounts. |
| 248 | 203 | * |
| 249 | 204 | * @param array $message |
| 250 | - * | |
| 251 | 205 | * @return bool |
| 252 | 206 | */ |
| 253 | 207 | private function is_for_user( $message ) { |
| 254 | - if ( FrmApiHelper::is_for_user( $message ) ) { | |
| 208 | + if ( ! isset( $message['who'] ) || $message['who'] === 'all' ) { | |
| 255 | 209 | return true; |
| 256 | 210 | } |
| 257 | - | |
| 258 | 211 | $who = (array) $message['who']; |
| 259 | - | |
| 212 | + if ( in_array( 'all', $who, true ) || in_array( 'everyone', $who, true ) ) { | |
| 213 | + return true; | |
| 214 | + } | |
| 215 | + if ( in_array( $this->get_user_type(), $who, true ) ) { | |
| 216 | + return true; | |
| 217 | + } | |
| 218 | + if ( in_array( 'free_first_30', $who, true ) && $this->is_free_first_30() ) { | |
| 219 | + return true; | |
| 220 | + } | |
| 260 | 221 | /** |
| 261 | 222 | * Allow for other special inbox cases in other add-ons. |
| 262 | 223 | * |
| 263 | 224 | * @since 6.8.1 |
| @@ -268,8 +229,16 @@ | ||
| 268 | 229 | */ |
| 269 | 230 | return (bool) apply_filters( 'frm_inbox_message_is_for_user', false, $who, $message ); |
| 270 | 231 | } |
| 271 | 232 | |
| 233 | + private function get_user_type() { | |
| 234 | + if ( ! FrmAppHelper::pro_is_installed() ) { | |
| 235 | + return 'free'; | |
| 236 | + } | |
| 237 | + | |
| 238 | + return FrmAddonsController::license_type(); | |
| 239 | + } | |
| 240 | + | |
| 272 | 241 | /** |
| 273 | 242 | * @param string $key |
| 274 | 243 | * |
| 275 | 244 | * @return void |
| @@ -290,20 +259,16 @@ | ||
| 290 | 259 | /** |
| 291 | 260 | * @since 4.05.02 |
| 292 | 261 | * |
| 293 | 262 | * @param string $key |
| 294 | - * | |
| 295 | 263 | * @return void |
| 296 | 264 | */ |
| 297 | 265 | public function mark_unread( $key ) { |
| 298 | 266 | $is_read = isset( self::$messages[ $key ] ) && isset( self::$messages[ $key ]['read'] ) && isset( self::$messages[ $key ]['read'][ get_current_user_id() ] ); |
| 299 | - | |
| 300 | - if ( ! $is_read ) { | |
| 301 | - return; | |
| 267 | + if ( $is_read ) { | |
| 268 | + unset( self::$messages[ $key ]['read'][ get_current_user_id() ] ); | |
| 269 | + $this->update_list(); | |
| 302 | 270 | } |
| 303 | - | |
| 304 | - unset( self::$messages[ $key ]['read'][ get_current_user_id() ] ); | |
| 305 | - $this->update_list(); | |
| 306 | 271 | } |
| 307 | 272 | |
| 308 | 273 | /** |
| 309 | 274 | * @param string $key |
| @@ -334,9 +299,8 @@ | ||
| 334 | 299 | * @return void |
| 335 | 300 | */ |
| 336 | 301 | private function dismiss_all() { |
| 337 | 302 | $user_id = get_current_user_id(); |
| 338 | - | |
| 339 | 303 | foreach ( self::$messages as $key => $message ) { |
| 340 | 304 | if ( ! isset( $message['dismissed'] ) ) { |
| 341 | 305 | self::$messages[ $key ]['dismissed'] = array(); |
| 342 | 306 | } |
| @@ -347,21 +311,16 @@ | ||
| 347 | 311 | } |
| 348 | 312 | $this->update_list(); |
| 349 | 313 | } |
| 350 | 314 | |
| 351 | - /** | |
| 352 | - * @return array | |
| 353 | - */ | |
| 354 | 315 | public function unread() { |
| 355 | 316 | $messages = $this->get_messages( 'filter' ); |
| 356 | 317 | $user_id = get_current_user_id(); |
| 357 | - | |
| 358 | 318 | foreach ( $messages as $t => $message ) { |
| 359 | 319 | if ( isset( $message['read'] ) && isset( $message['read'][ $user_id ] ) ) { |
| 360 | 320 | unset( $messages[ $t ] ); |
| 361 | 321 | } |
| 362 | 322 | } |
| 363 | - | |
| 364 | 323 | return $messages; |
| 365 | 324 | } |
| 366 | 325 | |
| 367 | 326 | /** |
| @@ -367,14 +326,12 @@ | ||
| 367 | 326 | /** |
| 368 | 327 | * @since 6.8.4 The $filtered parameter was added. |
| 369 | 328 | * |
| 370 | 329 | * @param bool $filtered |
| 371 | - * | |
| 372 | 330 | * @return string |
| 373 | 331 | */ |
| 374 | 332 | public function unread_html( $filtered = true ) { |
| 375 | 333 | $count = count( $this->unread() ); |
| 376 | - | |
| 377 | 334 | if ( ! $count ) { |
| 378 | 335 | return ''; |
| 379 | 336 | } |
| 380 | 337 | |
| @@ -395,18 +352,15 @@ | ||
| 395 | 352 | /** |
| 396 | 353 | * @since 4.05.02 |
| 397 | 354 | * |
| 398 | 355 | * @param string $key |
| 399 | - * | |
| 400 | 356 | * @return void |
| 401 | 357 | */ |
| 402 | 358 | public function remove( $key ) { |
| 403 | - if ( ! isset( self::$messages[ $key ] ) ) { | |
| 404 | - return; | |
| 359 | + if ( isset( self::$messages[ $key ] ) ) { | |
| 360 | + unset( self::$messages[ $key ] ); | |
| 361 | + $this->update_list(); | |
| 405 | 362 | } |
| 406 | - | |
| 407 | - unset( self::$messages[ $key ] ); | |
| 408 | - $this->update_list(); | |
| 409 | 363 | } |
| 410 | 364 | |
| 411 | 365 | /** |
| 412 | 366 | * @return void |
| @@ -411,21 +365,43 @@ | ||
| 411 | 365 | /** |
| 412 | 366 | * @return void |
| 413 | 367 | */ |
| 414 | 368 | private function update_list() { |
| 415 | - update_option( $this->option, self::$messages, false ); | |
| 369 | + update_option( $this->option, self::$messages, 'no' ); | |
| 416 | 370 | } |
| 417 | 371 | |
| 418 | 372 | /** |
| 373 | + * Check if user is still using the Lite version only, and within | |
| 374 | + * the first 30 days of activation. | |
| 375 | + * | |
| 376 | + * @since 6.16 | |
| 377 | + * | |
| 378 | + * @return bool | |
| 379 | + */ | |
| 380 | + private function is_free_first_30() { | |
| 381 | + $activation_timestamp = get_option( 'frm_first_activation' ); | |
| 382 | + if ( false === $activation_timestamp ) { | |
| 383 | + // If the option does not exist, assume that it is | |
| 384 | + // because the user was active before this option was introduced. | |
| 385 | + return false; | |
| 386 | + } | |
| 387 | + if ( FrmAppHelper::pro_is_included() ) { | |
| 388 | + // Not free. | |
| 389 | + return false; | |
| 390 | + } | |
| 391 | + $cutoff = strtotime( '-30 days' ); | |
| 392 | + return $activation_timestamp > $cutoff; | |
| 393 | + } | |
| 394 | + | |
| 395 | + /** | |
| 419 | 396 | * Show a banner message if one is available. |
| 420 | 397 | * |
| 421 | 398 | * @return bool True if a banner is available and shown. |
| 422 | 399 | */ |
| 423 | 400 | public static function maybe_show_banner() { |
| 424 | - if ( ! self::$banner_messages ) { | |
| 401 | + if ( empty( self::$banner_messages ) ) { | |
| 425 | 402 | return false; |
| 426 | 403 | } |
| 427 | - | |
| 428 | 404 | $message = end( self::$banner_messages ); |
| 429 | 405 | $cta = self::get_prepared_banner_cta( $message['cta'] ); |
| 430 | 406 | |
| 431 | 407 | require FrmAppHelper::plugin_path() . '/classes/views/inbox/banner.php'; |
| @@ -437,9 +413,8 @@ | ||
| 437 | 413 | * |
| 438 | 414 | * @since 6.8.4 |
| 439 | 415 | * |
| 440 | 416 | * @param string $cta |
| 441 | - * | |
| 442 | 417 | * @return string |
| 443 | 418 | */ |
| 444 | 419 | private static function get_prepared_banner_cta( $cta ) { |
| 445 | 420 | $cta = str_replace( 'button-secondary', 'button-primary', $cta ); |
| @@ -448,28 +423,25 @@ | ||
| 448 | 423 | /** |
| 449 | 424 | * Replace a single href attribute in the CTA. |
| 450 | 425 | * |
| 451 | 426 | * @param array $matches The regex results for a single match. |
| 452 | - * | |
| 453 | 427 | * @return string |
| 454 | 428 | */ |
| 455 | 429 | function ( $matches ) { |
| 456 | - $url = $matches[2]; | |
| 430 | + $url = $matches[2]; | |
| 431 | + $parts = parse_url( $url ); | |
| 457 | 432 | |
| 458 | 433 | if ( '#' === $url ) { |
| 459 | 434 | return 'href="#"'; |
| 460 | 435 | } |
| 461 | 436 | |
| 462 | - $parts = parse_url( $url ); | |
| 463 | 437 | $query = array(); |
| 464 | - | |
| 465 | 438 | if ( isset( $parts['query'] ) ) { |
| 466 | 439 | parse_str( $parts['query'], $query ); |
| 467 | 440 | } |
| 468 | - | |
| 469 | 441 | $query['utm_medium'] = 'banner'; |
| 470 | 442 | $parts['query'] = http_build_query( $query ); |
| 471 | - return 'href="' . esc_url( $parts['scheme'] . '://' . $parts['host'] . $parts['path'] . '?' . $parts['query'] ) . '"'; | |
| 443 | + return 'href="' . $parts['scheme'] . '://' . $parts['host'] . $parts['path'] . '?' . $parts['query'] . '"'; | |
| 472 | 444 | }, |
| 473 | 445 | $cta |
| 474 | 446 | ); |
| 475 | 447 | } |
| @@ -478,11 +450,10 @@ | ||
| 478 | 450 | * @return void |
| 479 | 451 | */ |
| 480 | 452 | public static function maybe_disable_screen_options() { |
| 481 | 453 | self::$banner_messages = self::get_banner_messages(); |
| 482 | - | |
| 483 | 454 | if ( self::$banner_messages ) { |
| 484 | - // Disable screen options tab when displaying banner messages because it gets in the way of the banner. | |
| 455 | + // disable screen options tab when displaying banner messages because it gets in the way of the banner. | |
| 485 | 456 | add_filter( 'screen_options_show_screen', '__return_false' ); |
| 486 | 457 | } |
| 487 | 458 | } |
| 488 | 459 | |
| @@ -511,9 +482,8 @@ | ||
| 511 | 482 | * |
| 512 | 483 | * @since 6.8.4 |
| 513 | 484 | * |
| 514 | 485 | * @param string $key The key we are checking for (ie. banner or slidein). |
| 515 | - * | |
| 516 | 486 | * @return array |
| 517 | 487 | */ |
| 518 | 488 | private static function get_messages_with_key( $key ) { |
| 519 | 489 | $inbox = new self(); |
| @@ -544,9 +514,8 @@ | ||
| 544 | 514 | * @return array|false |
| 545 | 515 | */ |
| 546 | 516 | public static function get_inbox_slide_in_value_for_js() { |
| 547 | 517 | $messages = self::get_slidein_messages(); |
| 548 | - | |
| 549 | 518 | if ( ! $messages ) { |
| 550 | 519 | return false; |
| 551 | 520 | } |
| 552 | 521 | |
| @@ -567,42 +536,11 @@ | ||
| 567 | 536 | |
| 568 | 537 | return array_reduce( |
| 569 | 538 | $keys_to_return, |
| 570 | 539 | function ( $total, $key ) use ( $message ) { |
| 571 | - $total[ $key ] = $message[ $key ] ?? ''; | |
| 540 | + $total[ $key ] = isset( $message[ $key ] ) ? $message[ $key ] : ''; | |
| 572 | 541 | return $total; |
| 573 | 542 | }, |
| 574 | 543 | array() |
| 575 | 544 | ); |
| 576 | - } | |
| 577 | - | |
| 578 | - /** | |
| 579 | - * Clear the inbox cache by deleting the associated option from the database. | |
| 580 | - * | |
| 581 | - * @since 6.17 | |
| 582 | - * | |
| 583 | - * @return void | |
| 584 | - */ | |
| 585 | - public static function clear_cache() { | |
| 586 | - delete_option( 'frm_inbox' ); | |
| 587 | - } | |
| 588 | - | |
| 589 | - /** | |
| 590 | - * Check inbox for an "error" type. This is displayed on the form list page if one exists. | |
| 591 | - * | |
| 592 | - * @since 6.19 | |
| 593 | - * | |
| 594 | - * @return array|false | |
| 595 | - */ | |
| 596 | - public static function check_for_error() { | |
| 597 | - $inbox = new self(); | |
| 598 | - $messages = $inbox->get_messages( 'filter' ); | |
| 599 | - | |
| 600 | - foreach ( $messages as $message ) { | |
| 601 | - if ( is_array( $message ) && isset( $message['type'] ) && 'error' === $message['type'] ) { | |
| 602 | - return $message; | |
| 603 | - } | |
| 604 | - } | |
| 605 | - | |
| 606 | - return false; | |
| 607 | 545 | } |
| 608 | 546 | } |