Use PascalCase for class names

Created on 28 July 2023, over 1 year ago

Updated 18 September 2024, 3 months ago

Problem/Motivation

Coming from 3228147 💬 Class 'Drupal\simple_sitemap\Queue\SimpleSitemapQueue' not found Fixed I stumbled across other possible breaking changes in the future. It affects submodule simple_sitemap_engines and testing but I think it would be better to be consistent here.

Proposed resolution

Renaming corresponding files using PascalCase pattern → SimplesitemapEnginesForm will be SimpleSitemapEnginesForm
Fix annotations and comments.

Remaining tasks

Add a note to the release notes about possible breaking changes (see 3228147 💬 Class 'Drupal\simple_sitemap\Queue\SimpleSitemapQueue' not found Fixed for possible effects), so changes should be not part of a patch release.

📌 Task

Status

Needs review

Version

4.0

Component

Code

Created by

🇩🇪Germany rwam

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @rwam
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MySQL 8
last update over 1 year ago
32 pass
@rwam opened merge request.
Status changed to Needs review over 1 year ago8:49am 28 July 2023
Comment over 1 year ago →
🇩🇪Germany rwam
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MySQL 8
last update over 1 year ago
32 pass
Comment 3 months ago →
🇷🇺Russia walkingdexter

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024