| @@ -301,8 +301,38 @@ | ||
| 301 | 301 | { |
| 302 | 302 | return Arr::get($this->checkout_data, 'is_locked') === 'yes' && $this->order_id; |
| 303 | 303 | } |
| 304 | 304 | |
| 305 | + /** | |
| 306 | + * Whether this cart can still take an additional item, such as an order bump. | |
| 307 | + * | |
| 308 | + * False when the cart is locked to an existing payment (custom payment link, | |
| 309 | + * renewal invoice, early installment) or already carries an upgrade. | |
| 310 | + * | |
| 311 | + * `is_locked` is a 'yes'/'no' string, so it must be compared explicitly — | |
| 312 | + * `!empty()` treats the string 'no' as locked. | |
| 313 | + * | |
| 314 | + * Deliberately distinct from isLocked(), which additionally requires order_id | |
| 315 | + * and is therefore false for renewal and early-installment carts, which never | |
| 316 | + * set that column. | |
| 317 | + * | |
| 318 | + * Filterable so an integration that locks its cart only to pin its own item | |
| 319 | + * (e.g. a booking) can still take order bumps. The filter only decides the | |
| 320 | + * lock; a cart carrying an upgrade is refused after it either way. The upgrade | |
| 321 | + * swap in WebCheckoutHandler::handleOrderBumpRequest() ignores this filter and | |
| 322 | + * keeps refusing any `is_locked` cart. | |
| 323 | + */ | |
| 324 | + public function acceptsAdditionalItems() | |
| 325 | + { | |
| 326 | + $accepts = (bool) apply_filters( | |
| 327 | + 'fluent_cart/cart/accepts_additional_items', | |
| 328 | + Arr::get($this->checkout_data, 'is_locked') !== 'yes', | |
| 329 | + ['cart' => $this] | |
| 330 | + ); | |
| 331 | + | |
| 332 | + return $accepts && empty(Arr::get($this->checkout_data, 'upgrade_data')); | |
| 333 | + } | |
| 334 | + | |
| 305 | 335 | public function addItem($item = [], $replacingIndex = null) |
| 306 | 336 | { |
| 307 | 337 | if ($this->isLocked()) { |
| 308 | 338 | return new \WP_Error('cart_locked', __('This cart is locked and cannot be modified.', 'fluent-cart')); |
| @@ -387,8 +417,12 @@ | ||
| 387 | 417 | |
| 388 | 418 | if ($quantity == 0) { |
| 389 | 419 | // that means we have to remove it |
| 390 | 420 | return $this->removeItem($variation->id, Arr::get($config, 'remove_args', []), true); |
| 421 | + } | |
| 422 | + | |
| 423 | + if (!$variation->product) { | |
| 424 | + return new \WP_Error('product_not_found', __('This product is no longer available.', 'fluent-cart')); | |
| 391 | 425 | } |
| 392 | 426 | |
| 393 | 427 | $validate = Arr::get($config, 'will_validate', false); |
| 394 | 428 | |
