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

Upgrade #5329

Merged
merged 50 commits into from
Apr 18, 2024
Merged

Upgrade #5329

merged 50 commits into from
Apr 18, 2024

Conversation

AngelFQC
Copy link
Member

@AngelFQC AngelFQC commented Apr 2, 2024

No description provided.

@AngelFQC AngelFQC force-pushed the upgrade branch 2 times, most recently from c177567 to 5c4ab05 Compare April 3, 2024 16:36
Copy link

codecov bot commented Apr 3, 2024

Codecov Report

Attention: Patch coverage is 16.64954% with 811 lines in your changes are missing coverage. Please review.

Project coverage is 40.03%. Comparing base (50ceb28) to head (6c79897).

❗ Current head 6c79897 differs from pull request most recent head 8bbdaab. Consider uploading reports for the commit 8bbdaab to get more accurate results

Files Patch % Lines
...e/Migrations/Schema/V200/Version20230913162700.php 0.00% 56 Missing ⚠️
...e/Migrations/Schema/V200/Version20231022124700.php 0.00% 56 Missing ⚠️
...e/Migrations/Schema/V200/Version20201212203625.php 0.00% 34 Missing ⚠️
...e/Migrations/Schema/V200/Version20201215160445.php 0.00% 34 Missing ⚠️
...c/CoreBundle/ServiceHelper/TicketProjectHelper.php 0.00% 32 Missing ⚠️
...e/Migrations/Schema/V200/Version20201217124011.php 0.00% 28 Missing ⚠️
...e/Migrations/Schema/V200/Version20201215142610.php 0.00% 25 Missing ⚠️
...e/Migrations/Schema/V200/Version20230216122900.php 0.00% 24 Missing ⚠️
...e/Migrations/Schema/V200/Version20201212195011.php 0.00% 19 Missing ⚠️
...e/Migrations/Schema/V200/Version20201215072918.php 0.00% 19 Missing ⚠️
... and 89 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5329      +/-   ##
============================================
+ Coverage     39.31%   40.03%   +0.72%     
- Complexity    10282    10313      +31     
============================================
  Files           814      817       +3     
  Lines         43555    43373     -182     
============================================
+ Hits          17125    17366     +241     
+ Misses        26430    26007     -423     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@chamilo chamilo deleted a comment from codeclimate bot Apr 3, 2024
/**
* @Route("/{category}/{path}", methods={"GET"}, requirements={"path"=".+"}, name="chamilo_core_asset_showfile")
*/
#[Route(path: '/{category}/{path}', methods: ['GET'], requirements: ['path' => '.+'], name: 'chamilo_core_asset_showfile')]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead.

* @Route("/resources/chat/", name="chat_home", options={"expose"=true})
*/
public function indexAction(Request $request): Response
#[Route(path: '/resources/chat/', name: 'chat_home', options: ['expose' => true])]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead.

*/
public function indexAction(Request $request): Response
#[Route(path: '/resources/chat/', name: 'chat_home', options: ['expose' => true])]
public function index(): Response
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must use "/**" style comments for a function comment

* @Route("/resources/chat/conversations/", name="chat_ajax", options={"expose"=true})
*/
public function ajaxAction(Request $request, ResourceNodeRepository $repo): Response
#[Route(path: '/resources/chat/conversations/', name: 'chat_ajax', options: ['expose' => true])]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead.

*/
public function ajaxAction(Request $request, ResourceNodeRepository $repo): Response
#[Route(path: '/resources/chat/conversations/', name: 'chat_ajax', options: ['expose' => true])]
public function ajax(Request $request, ResourceNodeRepository $repo): Response
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must use "/**" style comments for a function comment

/**
* @Route("/contact", name="contact")
*/
#[Route(path: '/contact', name: 'contact')]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead.

*/
public function editorTemplatesAction(TranslatorInterface $translator, RouterInterface $router): Response
#[Route(path: '/templates', methods: ['GET'], name: 'editor_templates')]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead.

*/
public function editorTemplatesAction(TranslatorInterface $translator, RouterInterface $router): Response
#[Route(path: '/templates', methods: ['GET'], name: 'editor_templates')]
public function editorTemplates(TranslatorInterface $translator, RouterInterface $router): Response
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must use "/**" style comments for a function comment

@@ -16,7 +16,7 @@

class ExceptionController extends AbstractController
{
public function showAction(Exception $exception): Response
public function show(Exception $exception): Response
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing function doc comment

* @Route("/error")
*/
public function errorAction(Request $request): Response
#[Route(path: '/error')]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead.

*/
public function errorAction(Request $request): Response
#[Route(path: '/error')]
public function error(Request $request): Response
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must use "/**" style comments for a function comment

@@ -43,7 +43,7 @@ public function list(): JsonResponse
}

#[Route('/upload', name: 'file_manager_upload', methods: ['POST'])]
public function upload(Request $request): JsonResponse
public function upload(): JsonResponse
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must use "/**" style comments for a function comment

@@ -81,21 +81,21 @@
}

#[Route('/create-folder', name: 'file_manager_create_folder', methods: ['POST'])]
public function createFolder(Request $request): JsonResponse
public function createFolder(): JsonResponse
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must use "/**" style comments for a function comment

{
// Implement logic to create new folders
return $this->json(['message' => 'Folder created successfully']);
}

#[Route('/rename', name: 'file_manager_rename', methods: ['POST'])]
public function rename(Request $request): JsonResponse
public function rename(): JsonResponse
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must use "/**" style comments for a function comment

{
// Implement logic to rename files/folders
return $this->json(['message' => 'File/folder renamed successfully']);
}

#[Route('/delete', name: 'file_manager_delete', methods: ['DELETE'])]
public function delete(Request $request): JsonResponse
public function delete(): JsonResponse
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must use "/**" style comments for a function comment

* @Route("/social", name="social", options={"expose"=true}, name="chamilo_core_socialnetwork")
* @Route("/admin", name="admin", options={"expose"=true})
*/
#[Route('/', name: 'index', options: ['expose' => true], methods: ['GET', 'POST'])]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead.

#[Route('/sessions', name: 'sessions')]
#[Route('/sessions/{extra}', name: 'sessions_options')]
#[Route('/admin/configuration/colors', name: 'configuration_colors')]
public function indexAction(): Response
#[Route('/home', name: 'home', options: ['expose' => true], methods: ['GET', 'POST'])]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead.

#[Route('/sessions', name: 'sessions')]
#[Route('/sessions/{extra}', name: 'sessions_options')]
#[Route('/admin/configuration/colors', name: 'configuration_colors')]
public function indexAction(): Response
#[Route('/home', name: 'home', options: ['expose' => true], methods: ['GET', 'POST'])]
#[Route('/login', name: 'login', options: ['expose' => true], methods: ['GET', 'POST'])]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead.

public function indexAction(): Response
#[Route('/home', name: 'home', options: ['expose' => true], methods: ['GET', 'POST'])]
#[Route('/login', name: 'login', options: ['expose' => true], methods: ['GET', 'POST'])]
#[Route('/faq', name: 'faq', options: ['expose' => true], methods: ['GET', 'POST'])]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead.

#[Route('/home', name: 'home', options: ['expose' => true], methods: ['GET', 'POST'])]
#[Route('/login', name: 'login', options: ['expose' => true], methods: ['GET', 'POST'])]
#[Route('/faq', name: 'faq', options: ['expose' => true], methods: ['GET', 'POST'])]
#[Route('/demo', name: 'demo', options: ['expose' => true], methods: ['GET', 'POST'])]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 5066 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 5070 lines exceeds the maximum allowed for the inline comments feature.

@AngelFQC AngelFQC force-pushed the upgrade branch 2 times, most recently from 6a485b1 to 04e1f54 Compare April 5, 2024 16:14
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 5335 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 5339 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 5346 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 5365 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 8092 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 8061 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 8007 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 9354 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 9362 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 11407 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 11411 lines exceeds the maximum allowed for the inline comments feature.

Copy link

codeclimate bot commented Apr 18, 2024

An error occurred when fetching issues.

View more on Code Climate.

@AngelFQC AngelFQC merged commit ce9491e into chamilo:master Apr 18, 2024
1 of 7 checks passed
@AngelFQC AngelFQC deleted the upgrade branch April 18, 2024 16:44
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.

1 participant