Skip to content

Fixed various deprecation fixes for Symfony >6.1, some code changes for PHP >= 8.0#435

Closed
joesenova wants to merge 9 commits into1up-lab:masterfrom
joesenova:updates/434-430-rework-for-php8-with-attributes
Closed

Fixed various deprecation fixes for Symfony >6.1, some code changes for PHP >= 8.0#435
joesenova wants to merge 9 commits into1up-lab:masterfrom
joesenova:updates/434-430-rework-for-php8-with-attributes

Conversation

@joesenova
Copy link
Copy Markdown

  1. Implemented the AsCommand attributes for final $defaultName
  2. Removed support for PHP <8.0
  3. Reworked the __construct methods to be on-par with php >=8.0
  4. Removed all rferences to GetMasterRequest - using only getMainRequest now
  5. Fixed custom_uploader.md documentation to use getMainRequest

We might need to bump the major package version due to dropping php v7 support with these changes

2. Removed support for PHP <8.0
3. Reworked the __construct methods to be on-par with php >=8.0
4. Removed all rferences to GetMasterRequest - using only getMainRequest now
5. Fixed custom_uploader.md documentation to use getMainRequest
@joesenova
Copy link
Copy Markdown
Author

There are some issues I am trying to solve here.

I have not got lots of knowledge with the github actions and the actions it runs.
If you know how to help me with this, please do let me know.

Thanks

@joesenova
Copy link
Copy Markdown
Author

Wait:

Here is codebase that has all the above fixed already.

https://github.com/dlxmedia/OneupUploaderBundle

@bytehead
Copy link
Copy Markdown
Member

bytehead commented Aug 7, 2023

Yes, that's the code from #432 (but closed unfortunately). I'll have look at yours 👍

@joesenova
Copy link
Copy Markdown
Author

Let me know what is needed here.

Thanks

@evertharmeling
Copy link
Copy Markdown
Contributor

evertharmeling commented Nov 23, 2023

Any update on this PR? As Symfony 7 is released shortly (end nov 2023) it would be nice to have this going forward and add support for Symfony 7.

If there's any help needed, let me know.

EDIT: I've created a fork based on this PR and added support for Symfony 7 in a sf-7-fork, just some minor adjustments! 🚀

@knallcharge
Copy link
Copy Markdown
Contributor

With Symfony 6.4 and 7.0 in plain sight, any chance this will be merged any time soon?

@bytehead
Copy link
Copy Markdown
Member

@evertharmeling can you make a PR from your branch to this PR?

@evertharmeling evertharmeling mentioned this pull request Nov 28, 2023
@evertharmeling
Copy link
Copy Markdown
Contributor

Not sure how I could add it to this PR, but made a new one #436

@bytehead
Copy link
Copy Markdown
Member

Closing this in favor of #436 (your precious work is included in there, thank you very much @joesenova!)

@bytehead bytehead closed this Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants