- 🇮🇳India Akshay kashyap
Created a patch for Drupal 10 compatibility fixes, please review.
Thank You. - Status changed to RTBC
over 1 year ago 7:04am 24 April 2023 - 🇮🇳India bunty oberoi
Hi @Akshay,
Verified and tested Patch Comment #4 on Drupal 10.1.x Applied patch cleanly
oberoi@oberoi-H510M-S2:~/Documents/projects/drupal/themes/contrib/moon_zymphonies_theme$ git apply -v Automated-Drupal-10-3297700-4.patch Checking patch moon_zymphonies_theme.info.yml... Checking patch moon_zymphonies_theme.theme... Applied patch moon_zymphonies_theme.info.yml cleanly. Applied patch moon_zymphonies_theme.theme cleanly.
Testing Steps:-
1. Install Drupal 10.1.x version.
2. git clone https://git.drupalcode.org/project/moon_zymphonies_theme.git
2. Applied Patch #4 /projects/drupal/themes/contrib/moon_zymphonies_theme
3. Checked the changes difference using git diff
4. Installed theme and set it as default.Testing result:
moon_zymphonies_theme Theme is compatible with Drupal 10.1.x Moving the issue to RTBCNote:- Here i have found some errors while checking the coding standards using a Drupal check.
oberoi@oberoi-H510M-S2:~/Documents/projects/drupal$ vendor/mglaman/drupal-check/drupal-check themes/contrib/moon_zymphonies_theme/
1/1 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%------ ---------------------------------------------------------------------
Line moon_zymphonies_theme.theme
------ ---------------------------------------------------------------------
175 Function file_create_url not found.
💡 Learn more at https://phpstan.org/user-guide/discovering-symbols
311 Constant FILE_STATUS_PERMANENT not found.
💡 Learn more at https://phpstan.org/user-guide/discovering-symbols
------ ---------------------------------------------------------------------[ERROR] Found 2 errors
Thanks for using drupal-check!
Consider sponsoring the development of the maintainers which make drupal-check possible:
- phpstan (ondrejmirtes): https://github.com/sponsors/ondrejmirtes
- phpstan-deprecation-rules (ondrejmirtes)): https://github.com/sponsors/ondrejmirtes
- phpstan-drupal (mglaman)): https://github.com/sponsors/mglaman
- drupal-check (mglaman)): https://github.com/sponsors/mglaman - Status changed to Needs work
over 1 year ago 7:05am 24 April 2023 - Status changed to Needs review
over 1 year ago 9:20am 24 April 2023 - 🇮🇳India bunty oberoi
Hi @Akshay,
I have created the patch for Drupal coding standards fixes please review this patch. - Status changed to RTBC
over 1 year ago 9:29am 24 April 2023 - 🇮🇳India Akshay kashyap
@Bunty oberoi
Thanks for the work
1/1 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100% [OK] No errors Thanks for using Drupal-check!
#8 comment fixed the Drupal coding standard now move to RTBC.