| @@ -67,9 +67,9 @@ | ||
| 67 | 67 | |
| 68 | 68 | $table_name = self::table_name(); |
| 69 | 69 | |
| 70 | 70 | // @codingStandardsIgnoreStart |
| 71 | - $payment = $wpdb->get_row( | |
| 71 | + return $wpdb->get_row( | |
| 72 | 72 | $wpdb->prepare( |
| 73 | 73 | "SELECT |
| 74 | 74 | p.*, e.user_id |
| 75 | 75 | FROM `{$wpdb->prefix}frm_{$table_name}` p |
| @@ -78,10 +78,8 @@ | ||
| 78 | 78 | $id |
| 79 | 79 | ) |
| 80 | 80 | ); |
| 81 | 81 | // @codingStandardsIgnoreEnd |
| 82 | - | |
| 83 | - return $payment; | |
| 84 | 82 | } |
| 85 | 83 | |
| 86 | 84 | /** |
| 87 | 85 | * Handle routing for deleting a payment. |
| @@ -116,13 +114,9 @@ | ||
| 116 | 114 | $allowed = array( 'payments', 'subscriptions' ); |
| 117 | 115 | $default = reset( $allowed ); |
| 118 | 116 | $name = FrmAppHelper::get_param( 'type', $default, 'get', 'sanitize_text_field' ); |
| 119 | 117 | |
| 120 | - if ( ! in_array( $name, $allowed, true ) ) { | |
| 121 | - $name = $default; | |
| 122 | - } | |
| 123 | - | |
| 124 | - return $name; | |
| 118 | + return in_array( $name, $allowed, true ) ? $name : $default; | |
| 125 | 119 | } |
| 126 | 120 | |
| 127 | 121 | /** |
| 128 | 122 | * @return FrmTransLitePayment|FrmTransLiteSubscription |
| @@ -127,8 +121,11 @@ | ||
| 127 | 121 | /** |
| 128 | 122 | * @return FrmTransLitePayment|FrmTransLiteSubscription |
| 129 | 123 | */ |
| 130 | 124 | private static function the_class() { |
| 131 | - $class_name = self::table_name() === 'subscriptions' ? 'FrmTransLiteSubscription' : 'FrmTransLitePayment'; | |
| 132 | - return new $class_name(); | |
| 125 | + if ( self::table_name() === 'subscriptions' ) { | |
| 126 | + return new FrmTransLiteSubscription(); | |
| 127 | + } | |
| 128 | + | |
| 129 | + return new FrmTransLitePayment(); | |
| 133 | 130 | } |
| 134 | 131 | } |