Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[12.x] Add missing methods to the Illuminate\Contracts\Container\Container interface #52543

Closed
wants to merge 345 commits into from
Closed

[12.x] Add missing methods to the Illuminate\Contracts\Container\Container interface #52543

wants to merge 345 commits into from

Conversation

subhamchbty
Copy link

  1. refresh($abstract, $target, $method)
  2. rebinding($abstract, Closure $callback)

Proposed Changes

The changes have been made to the src/Illuminate/Contracts/Container/Container.php file, where the new methods have been added to the Illuminate\Contracts\Container\Container interface.

Potential Impact

These changes are intended to be backward-compatible and should not break existing code that relies on the Illuminate\Contracts\Container\Container interface. They provide additional capabilities that can be optionally utilized by developers.

jasonmccreary and others added 21 commits August 13, 2024 12:44
* Add `forceDestroy` to `SoftDeletes`

* Update Model.php

---------

Co-authored-by: Taylor Otwell <[email protected]>
* feat: Add clear stopping point for recursion

* style: fix styleci

* Update MySqlSchemaState.php

* Update DatabaseMySqlSchemaStateTest.php

* Update MySqlSchemaState.php

---------

Co-authored-by: Taylor Otwell <[email protected]>
* Run prepareNestedBatches on append/prependToChain

* Cleanup leftover import

* Update chain method

* Update Queueable.php

---------

Co-authored-by: Taylor Otwell <[email protected]>
* fix db:show with counts option

* fix db:table

* add more info

* formatting

* fix connection name and table size

* Update ShowCommand.php

---------

Co-authored-by: Taylor Otwell <[email protected]>
* Constrain key when asserting database has against a model

* Use already constrained database has methods

* Using a model instance that isn't saved to the database should fail
* Added `between` tests

* Added `between` assertion

* pint

* pint

* formatting

* min max

---------

Co-authored-by: Taylor Otwell <[email protected]>
* Adds asset prefetching strategies

* Formatting

* Rename config

* Use low fetch priority

* formatting

* formatting

---------

Co-authored-by: Taylor Otwell <[email protected]>
* feat: support attributes in `app()->call()`

* feat: support attributes in route dependencies resolution

* style: apply fixes from style-ci
* feat: make HigherOrderCollectionProxy generic via template types

* Add type tests for HigherOrderCollectionProxy
…search()` and `multisearch()` functions (#51669)

* Update PromptsAssertionTest.php

* Update ConfiguresPrompts.php

* Update PromptsAssertionTest.php

* Add tests for search and multisearch assertions

* Formatting

* Formatting

* Add `expectsSearch` method

---------

Co-authored-by: Jess Archer <[email protected]>
@subhamchbty subhamchbty changed the title This pull request adds two new methods to the Illuminate\Contracts\Container\Container interface [11.x] Add methods to the Illuminate\Contracts\Container\Container interface Aug 21, 2024
@subhamchbty subhamchbty changed the title [11.x] Add methods to the Illuminate\Contracts\Container\Container interface [11.x] Add missing methods to the Illuminate\Contracts\Container\Container interface Aug 21, 2024
@Jubeki
Copy link
Contributor

Jubeki commented Aug 21, 2024

Hey, you can't make changes to an interface on a minor release, because there could be apps which implement the interface and would not be confronted with a breaking change.

These changes can only happen in major versions (master branch).

@subhamchbty
Copy link
Author

subhamchbty commented Aug 21, 2024

@Jubeki Shall I change the base branch then? Or can you please suggest what I should do?

@Jubeki
Copy link
Contributor

Jubeki commented Aug 21, 2024

Yeah changing the base branch to master would be okay, because that would be the branch for the next major version 12.x

@subhamchbty subhamchbty changed the base branch from 11.x to master August 21, 2024 10:49
@subhamchbty subhamchbty changed the title [11.x] Add missing methods to the Illuminate\Contracts\Container\Container interface [12.x] Add missing methods to the Illuminate\Contracts\Container\Container interface Aug 21, 2024
@subhamchbty
Copy link
Author

Hi @Jubeki, I've create another PR as you suggested.

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.