Draft
Conversation
- Add `return` to array expressions in bundles.php snippets - Add use statements for classes used in snippets - Add @var PHPDoc annotations for undefined variables - Fix method return types (void, typed returns) - Fix interface imports for form mapper interfaces - Fix wrong class paths (Float\Type\Value → Float\Value) - Fix TaxonomyEntry constructor usage - Fix Author FieldType to use new Author\Value() - Fix FilterResponseEvent → ResponseEvent in cache subscriber - Fix TranslatorInterface namespace (Component vs Contracts) - Fix payment criteria to use setQuery() instead of property access - Fix payment LogicalAnd/LogicalOr to use variadic args, not array - Fix PaymentQuery/PaymentMethodQuery initialization - Fix CartCreateStruct and EntryAddStruct constructor arguments - Add @phpstan-ignore for intentional pattern issues - Fix Novactive Solr class and add missing return in searchAction - Fix storefront controller to extend AbstractController Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add correct use statements for all field type Value classes - Fix class aliases: use namespace imports (X\Namespace as Alias) instead of class imports (X\Namespace\Value as Alias) so Alias\Value resolves correctly - Add @var annotations for undefined variables - Fix incorrect property names: $values (Keyword), $destinationContentIds (RelationList) - Fix incorrect method: loadContentTypeByIdentifier() instead of loadContentType() - Fix incorrect constant: Type::DEFAULT_EMPTY instead of DateAndTime::DEFAULT_EMPTY - Add return statements to expression-only snippets (countryfield, type_and_value) - Fix variable names: $checkboxValue->bool instead of $value->bool - Fix use statements: EmailAddress\Value instead of Type, then new Value() - Add @phpstan-ignore-next-line for conceptual read-only property assignments Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add missing use statements (Query, LocationQuery, ProductQuery, Aggregation, Range, URLQuery, Criterion, ShoppingListQuery) to all code blocks - Fix ObjectStateTermAggregation: remove incorrect Location\ namespace prefix - Fix product_attribute_aggregations: use correct class names (AttributeSelectionTermAggregation, AttributeIntegerRangeAggregation) - Fix Range type errors: use Range::ofInt(), Range::ofFloat(), Range::ofDateTime() factory methods for proper generic typing - Fix TimeRangeAggregation: use integer seconds instead of DateTime objects - Fix ProductQuery snippets: use setAggregations() instead of direct property access (aggregations is private on ProductQuery) - Fix taxonomyentryid ProductQuery snippet accordingly - Fix SectionId URL criterion: use int[] instead of string[] - Add \@var annotations for undefined $query, $currency, $customerGroup Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add return types to methods (void, array, typed returns) - Fix missing use statements (Bundle, ContainerBuilder, Request, Language, etc.) - Replace invalid `self $this` annotations with proper service @var annotations - Replace `$this->service` patterns with standalone $service variables in snippets - Fix NotificationHandlerInterface to TranslatableNotificationHandlerInterface - Fix MVCEvents::INTERACTIVE_LOGIN to SecurityEvents::INTERACTIVE_LOGIN - Fix InteractiveLoginEvent to Symfony's version - Fix HEADER_X_FORWARDED_ALL (removed in Symfony) with individual constants - Fix user_id() undefined function with $userId variable - Fix ContentTagInterface missing use statements - Fix X-Location-Id header int→string cast - Fix IbexaSetList use statement, suppress SensiolabsSetList - Add @phpstan-ignore for unavoidable null-access patterns - Add return statement to array expression in update_from_4.6.md - Fix RectorConfig use statement Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Preview of modified files: Too many files modified in a single PR, preview link list is skipped. (267 files > 100) |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.