Skip to content

Conversation

alipowerful7
Copy link
Contributor

@alipowerful7 alipowerful7 commented Sep 7, 2025

wrong docblock:

    /**
     * Specify middleware that should be removed from the resource routes.
     *
     * @param  array|string  $middleware
     * @return $this|array
     */
    public function withoutMiddleware($middleware)
    {
        $this->options['excluded_middleware'] = array_merge(
            (array) ($this->options['excluded_middleware'] ?? []), Arr::wrap($middleware)
        );

        return $this;
    }

right docblock:

    /**
     * Specify middleware that should be removed from the resource routes.
     *
     * @param  array|string  $middleware
     * @return $this
     */
    public function withoutMiddleware($middleware)
    {
        $this->options['excluded_middleware'] = array_merge(
            (array) ($this->options['excluded_middleware'] ?? []), Arr::wrap($middleware)
        );

        return $this;
    }

in two files:
src/Illuminate/Routing/PendingResourceRegistration.php
src/Illuminate/Routing/PendingSingletonResourceRegistration.php

@taylorotwell taylorotwell merged commit dd4fa26 into laravel:12.x Sep 7, 2025
65 checks passed
@alipowerful7 alipowerful7 deleted the refactor/remove-unused-array-from-docblock branch September 8, 2025 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants