Upgrade api-platform #6211

AngelFQC · 2025-04-08T23:14:44Z

No description provided.

ywarnier · 2025-04-21T10:06:33Z

When I issue a composer install in this branch, I get:

Executing script cache:clear [KO]
 [KO]
Script cache:clear returned with error code 1
!!  
!!  In ArrayNode.php line 327:
!!                                                                                 
!!    Unrecognized option "metadata_backward_compatibility_layer" under "api_plat  
!!    form". Available options are "asset_package", "collection", "defaults", "de  
!!    scription", "docs_formats", "doctrine", "doctrine_mongodb_odm", "eager_load  
!!    ing", "elasticsearch", "enable_docs", "enable_entrypoint", "enable_link_sec  
!!    urity", "enable_profiler", "enable_re_doc", "enable_swagger", "enable_swagg  
!!    er_ui", "error_formats", "event_listeners_backward_compatibility_layer", "e  
!!    xception_to_status", "formats", "graphql", "handle_symfony_errors", "http_c  
!!    ache", "inflector", "jsonschema_formats", "keep_legacy_inflector", "maker",  
!!     "mapping", "mercure", "messenger", "name_converter", "oauth", "openapi", "  
!!    patch_formats", "path_segment_name_generator", "resource_class_directories"  
!!    , "serializer", "show_webby", "swagger", "title", "use_deprecated_json_sche  
!!    ma_type_factory", "use_symfony_listeners", "validator", "version".           
!!                                                                                 
!!  
!!  2025-04-21T10:05:22+00:00 [info] User Deprecated: Method "Symfony\Component\HttpKernel\Bundle\Bundle::build()" might add "void" as a native return type declaration in the future. Do the same in child class "Sensio\Bundle\FrameworkExtraBundle\SensioFrameworkExtraBundle" now to avoid errors or add an explicit @return annotation to suppress this message.
!!  
Script @auto-scripts was called via post-install-cmd

(just in case you have something different)

codeclimate · 2025-04-22T23:01:01Z

src/CourseBundle/Entity/CToolIntro.php

@@ -26,7 +26,22 @@
 use Symfony\Component\Uid\Uuid;
 use Symfony\Component\Validator\Constraints as Assert;

-#[ApiResource(operations: [new Get(security: 'is_granted(\'VIEW\', object)'), new Put(security: 'is_granted(\'EDIT\', object)'), new Delete(security: 'is_granted(\'DELETE\', object)'), new GetCollection(security: 'is_granted(\'ROLE_USER\')'), new Post(securityPostDenormalize: 'is_granted(\'CREATE\', object)')], security: 'is_granted(\'ROLE_ADMIN\') or is_granted(\'ROLE_CURRENT_COURSE_TEACHER\')', denormalizationContext: ['groups' => ['c_tool_intro:write']], normalizationContext: ['groups' => ['c_tool_intro:read']])]
+#[ApiResource(


Similar blocks of code found in 2 locations. Consider refactoring.

… and improve operation type check

…ntation

Replaced the deprecated ApiPlatform namespace with the updated one. This change ensures compatibility with the latest version of the ApiPlatform library and removes potential warnings.

…odern Doctrine methods

Replaced all Symfony Doctrine ORM annotations with PHP 8 attributes to modernize the codebase and improve readability. Additionally, updated type hints and property declarations to enforce strict typing and better align with PHP 8 standards.

…iver' fields in results

…ic type parameters.

…zAnswer classes

codeclimate · 2025-04-23T22:23:37Z

Code Climate has analyzed commit 15b4e9e and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category	Count
Duplication	2

View more on Code Climate.

AngelFQC force-pushed the upgrade-api branch 2 times, most recently from dc6b40d to 23d32c5 Compare April 22, 2025 22:34

codeclimate bot reviewed Apr 22, 2025

View reviewed changes

AngelFQC force-pushed the upgrade-api branch 3 times, most recently from 4a30071 to 311defd Compare April 23, 2025 21:22

AngelFQC added 12 commits April 23, 2025 17:14

Vendor: Bump api-platform to v3.0

6748e5b

Internal: API: remove metadata_backward_compatibility_layer config param

86495a4

Internal: API: Use new Filter class for API resources

4088d0e

Internal: API: Update provide method signature to include return type…

60dc75e

… and improve operation type check

Internal: API: Refactor data transformers to remove interface impleme…

7c62052

…ntation

CI: Update ApiTestCase namespace in AbstractApiTest.php

bef6663

Replaced the deprecated ApiPlatform namespace with the updated one. This change ensures compatibility with the latest version of the ApiPlatform library and removes potential warnings.

Internal: refactor syncronize_user_base_from_ldap.php script to use m…

ffcd6cf

…odern Doctrine methods

Internal: Refactor API Resource definitions for CTool and CToolIntro

8eb317a

CI: Fix SocialPort test removing unused 'userReceiver' and 'groupRece…

30be9fd

…iver' fields in results

Internal: Add @implements ProcessorInterface specifying their gener…

c0fbc53

…ic type parameters.

Internal: Remove unused dynamic properties in CForumCategory and CQui…

15b4e9e

…zAnswer classes

AngelFQC force-pushed the upgrade-api branch from 311defd to 15b4e9e Compare April 23, 2025 22:19

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Upgrade api-platform #6211

Upgrade api-platform #6211

AngelFQC commented Apr 8, 2025

ywarnier commented Apr 21, 2025

codeclimate bot Apr 22, 2025

codeclimate bot commented Apr 23, 2025

Upgrade api-platform #6211

Are you sure you want to change the base?

Upgrade api-platform #6211

Conversation

AngelFQC commented Apr 8, 2025

ywarnier commented Apr 21, 2025

codeclimate bot Apr 22, 2025

Choose a reason for hiding this comment

codeclimate bot commented Apr 23, 2025