-
Notifications
You must be signed in to change notification settings - Fork 0
PHP 8+ Support #94
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
Open
MouseEatsCat
wants to merge
67
commits into
main
Choose a base branch
from
feature/php8
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
PHP 8+ Support #94
Changes from all commits
Commits
Show all changes
67 commits
Select commit
Hold shift + click to select a range
27efeb5
Kind of working
MouseEatsCat 6f60f43
Functional errors
MouseEatsCat 5a01b80
Basic functioning admin
MouseEatsCat ff1059c
Patch usage of getParams + getParam
MouseEatsCat 3b60d6f
Support php 8.4
MouseEatsCat 72b966f
fix deprecation warnings
MouseEatsCat 203a6da
Fix csv translations with blank rows
MouseEatsCat cd4acec
Fix view services
MouseEatsCat 1ff298a
Tests - Fix container usages
MouseEatsCat b6fb75c
Fix container usage within admin
MouseEatsCat 0719156
Various admin fixes
MouseEatsCat 87233e5
Fix elfinder + container references
MouseEatsCat 13ceb04
Add System Info Template
MouseEatsCat 361bb3c
DDEV update php version to 8.5
MouseEatsCat 6dac489
Update Elfinder + ACL
MouseEatsCat 56ec735
Replace Deprecated: PDO::MYSQL_ATTR_INIT_COMMAND
MouseEatsCat 976a76e
System Info - Add charcoal version
MouseEatsCat 5d46680
DatabaseServiceProvider - add fallback for php < 8.4
MouseEatsCat f0ff9b9
System Info - Clean up values + interface
MouseEatsCat 58f1c15
System Info - add twig + mustache
MouseEatsCat 1ca6d98
Translation - isRendered - replace dynamic property with method
MouseEatsCat c287c0f
Cleanup charcoal bin
MouseEatsCat 5550e01
Github Actions - update php version
MouseEatsCat 183d633
Update lock file
MouseEatsCat b18a11f
GitHub Actions - Use PHP 8.4
MouseEatsCat c0a940f
Update requirements + workflows
MouseEatsCat 5accd3f
UI - fix tests container usage
MouseEatsCat 0479722
Update symfony + phpunit
MouseEatsCat 514c206
Monorepo Builder - replace packages-json with alternative
MouseEatsCat 1515ab7
Remove deprecated ReflectionProperty::setAccessible()
MouseEatsCat df77890
Update phpunit.xml configurations to v11
MouseEatsCat e44122d
Ignore test logs
MouseEatsCat d423a45
Translator - repair tests
MouseEatsCat 0832711
Translator - add backwards compatibility for older translation patterns
MouseEatsCat 9387b38
UI - add coverage classes
MouseEatsCat be30b6a
Translator - fallback for missing message_selector
MouseEatsCat 8d5f4a4
Core - fix tests
MouseEatsCat efda060
Core - Expression - Make sure name and active are serialized
MouseEatsCat 008c984
Object - add coverage + fix tests
MouseEatsCat b4fbd1b
Property - add coverage
MouseEatsCat 7cfe3b4
Property - fix tests
MouseEatsCat 6bd9e03
Config - Add coverage
MouseEatsCat 6e64b83
Config - fix tests
MouseEatsCat 72e86f9
Factory - add coverage
MouseEatsCat 01da20a
Cache - fix tests
MouseEatsCat f83e3d4
Admin - add coverage
MouseEatsCat c257f56
Admin - fix tests
MouseEatsCat bc9b443
Email - Fix tests
MouseEatsCat 393b189
DatabaseServiceProvider - Don't set MYSQL_ATTR_INIT_COMMAND on non-mysql
MouseEatsCat f874dd8
User - add coverage
MouseEatsCat 0745030
Remove empty tests
MouseEatsCat 88f085d
CMS - fix tests
MouseEatsCat 79ab63d
Image - add coverage + fix tests
MouseEatsCat 8441f50
App - add coverage
MouseEatsCat ab71183
App - fix tests
MouseEatsCat 5b7e831
Queue - remove empty tests
MouseEatsCat 9d3fcc0
View - fix tests
MouseEatsCat 83f06f7
Update composer using php 8.3
MouseEatsCat 9b33ebd
PHPCS - fix all errors
MouseEatsCat 308cd5b
PHPUnit - fix tests
MouseEatsCat c8cd8b8
Add base test for view Twig tests
MouseEatsCat a15d8ab
Mustache - TranslatorHelpers - fix return type
MouseEatsCat 543287a
System info - table add light class
MouseEatsCat 093bd11
Attempt - fix phpunit covers errors
MouseEatsCat 37354f3
PHPUnit - replace docblock @covers with Attributes
MouseEatsCat 8d32404
Mustache - TranslatorHelpers add string and Stringable as reutnr types
MouseEatsCat d807c5a
PHPUnit - update remaining docblock @covers
MouseEatsCat File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| name: charcoal | ||
| type: php | ||
| docroot: "" | ||
| php_version: "8.3" | ||
| webserver_type: apache-fpm | ||
| xdebug_enabled: false | ||
| additional_hostnames: [] | ||
| additional_fqdns: [] | ||
| database: | ||
| type: mariadb | ||
| version: "10.11" | ||
| use_dns_when_possible: true | ||
| composer_version: "2" | ||
| web_environment: [] | ||
| corepack_enable: false |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,6 @@ | ||
| .phpunit.result.cache | ||
| .phpunit.cache | ||
| tests/logs/* | ||
| charcoal_test | ||
| node_modules/ | ||
| packages/**/composer.lock | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Changes in requirements will need to be propagated to |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this just assign the array itself?