Problem/Motivation

This is a subset of the descriptions originally included in #2570985: Update the module descriptions on the Extend page to make the review easier.

The description of a module on the Extend page, describes what the module does.
Not all module descriptions on the Extend page follow the common format, as described in the Help text standard.
Module descriptions were reviewed and changed for Drupal 8, The module description on the Extend page, should follow the wording on the module's help page because these have been reviewed previously.

Proposed resolution

Change the descriptions in the modules .info.yml files for the stable modules so that they are correct, consistent, follow the same format, and fit with the wording used in the module's Help text.

Several modules were added to core in since this issue was started. If a module description is not in this patch consider opening a new issue and avoid scope creep.

Remaining tasks

Review the patch and make sure that all the changed help pages are correct, consistent, follow the same format, and fit with the wording used in the module's Help text, at /admin/help/module-name

User interface changes

This is a UI text change.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#93 interdiff-88-93.txt2.74 KBnod_
#93 core-3060616-93.patch18.56 KBnod_
#88 interdiff-81-88.txt363 bytesnod_
#88 core-3060616-88-1.patch16.6 KBnod_
#87 3060616-87.patch18.65 KBelber
#81 3060616-68.patch17 KBquietone
#81 interdiff-68-79.txt1.39 KBquietone
#81 diff-68-75.txt7.11 KBquietone
#79 interdiff_78-79.txt401 bytesranjith_kumar_k_u
#79 3060616-79.patch16.91 KBranjith_kumar_k_u
#78 3060616-78.patch17.04 KBMunavijayalakshmi
#76 3060616-75.patch16.61 KBMunavijayalakshmi
#72 beforepatch.png95.72 KBRinku Jacob 13
#72 afterpatch.png83.99 KBRinku Jacob 13
#68 3060616-68.patch17 KBelber
#65 interdiff_60-65.txt89.5 KBelber
#65 3060616.patch106.26 KBelber
#60 diff_46-60.txt1.61 KBkostyashupenko
#60 3060616-60.patch16.11 KBkostyashupenko
#56 after--patch--rest.png9.5 KBvikashsoni
#56 after--patch--block.png9.48 KBvikashsoni
#56 after--patch--ckeditor.png6.49 KBvikashsoni
#56 before--patch--ckeditor.png8.25 KBvikashsoni
#56 before--patch--block.png10.72 KBvikashsoni
#56 before--patch--rest.png9.89 KBvikashsoni
#50 Screenshot from 2021-06-22 11-21-44.png198.44 KBRinku Jacob 13
#46 interdiff_41-46.txt792 bytesNeslee Canil Pinto
#46 3060616-46.patch15.98 KBNeslee Canil Pinto
#43 Screenshot 2021-05-01 at 6.43.11 PM.png193.29 KBBhumikaVarshney
#43 after-patch.png20.68 KBBhumikaVarshney
#43 before-patch.png19.73 KBBhumikaVarshney
#41 interdiff-30-41.txt762 bytespaulocs
#41 3060616-41.patch14.98 KBpaulocs
#38 3060616-38.patch15.37 KBAbhijith S
#32 Aftr_patch_restful.png18.78 KBsonam.chaturvedi
#32 Bef_patch_restful.png18.58 KBsonam.chaturvedi
#32 Aftr_patch_ckedt.png13 KBsonam.chaturvedi
#32 Aftr_patch_block.png16.28 KBsonam.chaturvedi
#32 Bef_patch_ckedtr.png14.1 KBsonam.chaturvedi
#32 bef_patch_block.png18.01 KBsonam.chaturvedi
#30 interdiff-30.txt2.58 KBKapilV
#30 3060616-30.patch15.42 KBKapilV
#28 After_patch#26.png18.58 KBsonam.chaturvedi
#26 interdiff_22-26.txt2.39 KBadityasingh
#26 3060616-26.patch14.81 KBadityasingh
#24 Screenshot from 2021-02-02 15-31-43.png6.91 KBpaulocs
#22 3060616-22.patch15.1 KBnitesh624
#18 3060616-18.patch15.54 KBadamzimmermann
#13 3060616-13.patch15.56 KBmmatsoo
#11 3060616-11.patch15.54 KBravi.shankar
#8 3060616-8.patch15.37 KBswatichouhan012
#4 3060616-module-description-changes-4.patch16.07 KBifrik

Issue fork drupal-3060616

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ifrik created an issue. See original summary.

ifrik’s picture

I'm working on this during DevDays.

ifrik’s picture

Modules that should be covered in this issue: Blocks, CKEditor, Date time, Text editor, Field, Filter, Forum, Help, History, Link, Interface translation, Menu UI, Migrate Drupal, Node, Option, Path, Quick Edit, RDF, Responsive Image, Restful Webservices, Serialization, Shortcut, Testing, Statistics, Syslog, System, Text, Toolbar, Activity tracker, Update Manager, User, Views

ifrik’s picture

Status: Needs work » Needs review
FileSize
16.07 KB

This patch changes the module descriptions for the modules listed above, to ensure that they fit the formal criteria of forming a proper sentence together with the module name, and to use the wording of the module help texts which have been reviewed previously.
To review this, the module description can be compared to the beginning of the Help page for each module.

The module description of the Node module is used in a test, therefore this is also changed.

Version: 8.8.x-dev » 8.9.x-dev

Drupal 8.8.0-alpha1 will be released the week of October 14th, 2019, which means new developments and disruptive changes should now be targeted against the 8.9.x-dev branch. (Any changes to 8.9.x will also be committed to 9.0.x in preparation for Drupal 9’s release, but some changes like significant feature additions will be deferred to 9.1.x.). For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

alexpott’s picture

Version: 8.9.x-dev » 9.0.x-dev
Status: Needs review » Needs work

Almost certainly this patch needs a reroll for D9.

swatichouhan012’s picture

Assigned: Unassigned » swatichouhan012
swatichouhan012’s picture

Assigned: swatichouhan012 » Unassigned
Status: Needs work » Needs review
FileSize
15.37 KB

Hii @alexpott, i have rerolled patch for 9.0.x-dev, Kindly review.

xjm’s picture

Version: 9.0.x-dev » 9.1.x-dev

These string changes would be a minor-only. Since 8.9.x and 9.0.x are now in beta, I'm moving this to 9.1.x. Thanks!

xjm’s picture

Status: Needs review » Needs work

There's some syntax error in #8; maybe a quoting delimiter problem somewhere.

ravi.shankar’s picture

Status: Needs work » Needs review
FileSize
15.54 KB

Here I have again reroll patch #4.

Version: 9.1.x-dev » 9.2.x-dev

Drupal 9.1.0-alpha1 will be released the week of October 19, 2020, which means new developments and disruptive changes should now be targeted for the 9.2.x-dev branch. For more information see the Drupal 9 minor version schedule and the Allowed changes during the Drupal 9 release cycle.

mmatsoo’s picture

Re-rolled patch from #11 against 9.2.x.

adamzimmermann’s picture

I can confirm that the latest patch applies cleanly on the 9.2.x branch.

I'm not 100% sure how to confirm this is RTBC. Do we need to compare the help text page and the info file text again?

mmatsoo’s picture

@adamzimmermann When I initially re-rolled the patch, I reviewed each description text and found it was of good quality. (Messaging, grammar, spelling.)

Truly, I did not compare those texts to the modules' help text. Looking now, I see that the module help text is often much longer. Some examples for CKEditor and Datetime:

CKEditor Help Text

"The CKEditor module provides a highly-accessible, highly-usable visual text editor and adds a toolbar to text fields. Users can use buttons to format content and to create semantically correct and valid HTML. The CKEditor module uses the framework provided by the Text Editor module. It requires JavaScript to be enabled in the browser."

vs

ckeditor.info.yml description in patch

"Provides a visual text editor (WYSIWYG) and adds a toolbar to text fields using CKEditor."

Datetime Help Text

"The Datetime module provides a Date field that stores dates and times. It also provides the Form API elements datetime and datelist for use in programming modules. See the Field module help and the Field UI module help pages for general information on fields and how to create and manage them."

vs

datetime.info.yml description in patch

"Defines field types for storing dates and times."

I personally like having these succinct descriptions in the info.yml files. I could be swayed, but I do not really see any benefit in copying the help text, word for word. Not sure who makes the final decision.

mmatsoo’s picture

Status: Needs review » Reviewed & tested by the community
catch’s picture

Status: Reviewed & tested by the community » Needs work

Patch doesn't apply, but also noticed some things:

  1. -description: "WYSIWYG editing for rich text fields using CKEditor."
    +description: 'Provides a visual text editor (WYSIWYG) and adds a toolbar to text fields using CKEditor.'
    

    This line duplicates itself - the visual text editor and the 'toolbar to text fields' are the same thing.

  2. -description: 'Provides discussion forums.'
    +description: 'Allows users to create threaded discussion forums.'
    

    The forum module doesn't 'allow users to create threaded discussion forums' - it provides a discussion forum, within which users can create threaded discussions, so the original language is more accurate and precise.

  3. -description: 'Allows administrators to customize the site navigation menu.'
    +description: 'Provides a user interface for managing menus.'
    

    Many of the other items are changing language from "Provides a [thing]" to "Allows [someone] to do [thing]".

    fwiw I prefer the "Provides a [thing]" language since it's more direct, so I think it would be better to consistently move towards the shorter more direct language here.

adamzimmermann’s picture

Re-rolled the patch. If there is a consensus around the description syntax I'm happy to help with that.

FWIW the "allows" syntax is in one of the examples, however I'm personally always a fan of short direct descriptions.

Content Translation Allows users to translate content entities.

https://www.drupal.org/node/632280#s-1-short-description

ifrik’s picture

The wording "Allows users... " was explicitely choosen when we rewrote the help texts and these short descriptions for those case where the modules makes it possible for users (including site builders) to set up something on the site, rather then providing something out of the box.

The forum module does not provide a forum - it provides the functionality for site builders to create one or several, which other users can then use. The difference is usually visible in a minimal site installation rather then a default.

Another example where the difference might be clearer is the Taxonomy module: It just provides the option for users to create vocabularies and terms, but it does not provide a specific vocabulary - even though the Default installation also adds a Tags vocabulary during the installation process.

nitesh624’s picture

Assigned: Unassigned » nitesh624

Assigining myself working on it will update in few hours

nitesh624’s picture

nitesh624’s picture

Assigned: nitesh624 » Unassigned
Status: Needs work » Needs review
FileSize
15.1 KB
Abhijith S’s picture

Patch can't be applied,shows error.

Checking patch core/modules/block/block.info.yml...
error: while searching for:
name: Block
type: module
description: 'Controls the visual building blocks a page is constructed with. Blocks are boxes of content rendered into an area, or region, of a web page.'
package: Core
version: VERSION
configure: block.admin_display

error: patch failed: core/modules/block/block.info.yml:1
error: core/modules/block/block.info.yml: patch does not apply
Checking patch core/modules/ckeditor/ckeditor.info.yml...
error: while searching for:
name: CKEditor
type: module
description: "WYSIWYG editing for rich text fields using CKEditor."
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/ckeditor/ckeditor.info.yml:1
error: core/modules/ckeditor/ckeditor.info.yml: patch does not apply
Checking patch core/modules/datetime/datetime.info.yml...
error: while searching for:
name: Datetime
type: module
description: Defines datetime form elements and a datetime field type.
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/datetime/datetime.info.yml:1
error: core/modules/datetime/datetime.info.yml: patch does not apply
Checking patch core/modules/editor/editor.info.yml...
error: while searching for:
name: 'Text Editor'
type: module
description: 'Provides a means to associate text formats with text editor libraries such as WYSIWYGs or toolbars.'
package: Core
version: VERSION
dependencies:
  - drupal:filter

error: patch failed: core/modules/editor/editor.info.yml:1
error: core/modules/editor/editor.info.yml: patch does not apply
Checking patch core/modules/field/field.info.yml...
error: while searching for:
name: Field
type: module
description: 'Field API to add fields to entities like nodes and users.'
package: Core
version: VERSION

error: patch failed: core/modules/field/field.info.yml:1
error: core/modules/field/field.info.yml: patch does not apply
Checking patch core/modules/filter/filter.info.yml...
error: while searching for:
name: Filter
type: module
description: 'Filters content in preparation for display.'
package: Core
version: VERSION
configure: filter.admin_overview

error: patch failed: core/modules/filter/filter.info.yml:1
error: core/modules/filter/filter.info.yml: patch does not apply
Checking patch core/modules/help/help.info.yml...
error: while searching for:
name: Help
type: module
description: 'Manages the display of online help.'
package: Core
version: VERSION

error: patch failed: core/modules/help/help.info.yml:1
error: core/modules/help/help.info.yml: patch does not apply
Checking patch core/modules/history/history.info.yml...
error: while searching for:
name: History
type: module
description: 'Records which user has read which content.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/history/history.info.yml:1
error: core/modules/history/history.info.yml: patch does not apply
Checking patch core/modules/link/link.info.yml...
error: while searching for:
name: Link
type: module
description: 'Provides a simple link field type.'
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/link/link.info.yml:1
error: core/modules/link/link.info.yml: patch does not apply
Checking patch core/modules/locale/locale.info.yml...
error: while searching for:
name: 'Interface Translation'
type: module
description: 'Translates the built-in user interface.'
configure: locale.translate_page
package: Multilingual
version: VERSION

error: patch failed: core/modules/locale/locale.info.yml:1
error: core/modules/locale/locale.info.yml: patch does not apply
Checking patch core/modules/menu_ui/menu_ui.info.yml...
error: while searching for:
name: Menu UI
type: module
description: 'Allows administrators to customize the site navigation menu.'
package: Core
version: VERSION
configure: entity.menu.collection

error: patch failed: core/modules/menu_ui/menu_ui.info.yml:1
error: core/modules/menu_ui/menu_ui.info.yml: patch does not apply
Checking patch core/modules/migrate_drupal/migrate_drupal.info.yml...
error: while searching for:
name: Migrate Drupal
type: module
description: 'Contains migrations from older Drupal versions.'
package: Migration
version: VERSION
dependencies:

error: patch failed: core/modules/migrate_drupal/migrate_drupal.info.yml:1
error: core/modules/migrate_drupal/migrate_drupal.info.yml: patch does not apply
Checking patch core/modules/node/node.info.yml...
error: while searching for:
name: Node
type: module
description: 'Allows content to be submitted to the site and displayed on pages.'
package: Core
version: VERSION
configure: entity.node_type.collection

error: patch failed: core/modules/node/node.info.yml:1
error: core/modules/node/node.info.yml: patch does not apply
Checking patch core/modules/options/options.info.yml...
error: while searching for:
name: Options
type: module
description: 'Defines selection, check box and radio button widgets for text and numeric fields.'
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/options/options.info.yml:1
error: core/modules/options/options.info.yml: patch does not apply
Checking patch core/modules/path/path.info.yml...
error: while searching for:
name: Path
type: module
description: 'Allows users to rename URLs.'
package: Core
version: VERSION
configure: entity.path_alias.collection

error: patch failed: core/modules/path/path.info.yml:1
error: core/modules/path/path.info.yml: patch does not apply
Checking patch core/modules/quickedit/quickedit.info.yml...
error: while searching for:
name: Quick Edit
type: module
description: 'In-place content editing.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/quickedit/quickedit.info.yml:1
error: core/modules/quickedit/quickedit.info.yml: patch does not apply
Checking patch core/modules/rdf/rdf.info.yml...
error: while searching for:
name: RDF
type: module
description: 'Enriches your content with metadata to let other applications (e.g. search engines, aggregators) better understand its relationships and attributes.'
package: Core
version: VERSION

error: patch failed: core/modules/rdf/rdf.info.yml:1
error: core/modules/rdf/rdf.info.yml: patch does not apply
Checking patch core/modules/responsive_image/responsive_image.info.yml...
error: while searching for:
name: Responsive Image
type: module
description: 'Provides an image formatter and breakpoint mappings to output responsive images using the HTML5 picture tag.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/responsive_image/responsive_image.info.yml:1
error: core/modules/responsive_image/responsive_image.info.yml: patch does not apply
Checking patch core/modules/serialization/serialization.info.yml...
error: while searching for:
name: Serialization
type: module
description: 'Provides a service for (de)serializing data to/from formats such as JSON and XML.'
package: Web services
version: VERSION

error: patch failed: core/modules/serialization/serialization.info.yml:1
error: core/modules/serialization/serialization.info.yml: patch does not apply
Checking patch core/modules/shortcut/shortcut.info.yml...
error: while searching for:
name: Shortcut
type: module
description: 'Allows users to manage customizable lists of shortcut links.'
package: Core
version: VERSION
configure: entity.shortcut_set.collection

error: patch failed: core/modules/shortcut/shortcut.info.yml:1
error: core/modules/shortcut/shortcut.info.yml: patch does not apply
Checking patch core/modules/statistics/statistics.info.yml...
error: while searching for:
name: Statistics
type: module
description: 'Logs content statistics for your site.'
package: Core
version: VERSION
configure: statistics.settings

error: patch failed: core/modules/statistics/statistics.info.yml:1
error: core/modules/statistics/statistics.info.yml: patch does not apply
Checking patch core/modules/syslog/syslog.info.yml...
error: while searching for:
name: Syslog
type: module
description: 'Logs and records system events to syslog.'
package: Core
version: VERSION
configure: system.logging_settings

error: patch failed: core/modules/syslog/syslog.info.yml:1
error: core/modules/syslog/syslog.info.yml: patch does not apply
Checking patch core/modules/system/system.info.yml...
error: while searching for:
name: System
type: module
description: 'Handles general site configuration for administrators.'
package: Core
version: VERSION
required: true

error: patch failed: core/modules/system/system.info.yml:1
error: core/modules/system/system.info.yml: patch does not apply
Checking patch core/modules/system/tests/src/Functional/Module/PrepareUninstallTest.php...
Checking patch core/modules/text/text.info.yml...
error: while searching for:
name: Text
type: module
description: 'Defines simple text field types.'
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/text/text.info.yml:1
error: core/modules/text/text.info.yml: patch does not apply
Checking patch core/modules/toolbar/toolbar.info.yml...
error: while searching for:
name: Toolbar
type: module
description: 'Provides a toolbar that shows the top-level administration menu links and links from other modules.'
package: Core
version: VERSION
dependencies:
  - drupal:breakpoint

error: patch failed: core/modules/toolbar/toolbar.info.yml:1
error: core/modules/toolbar/toolbar.info.yml: patch does not apply
Checking patch core/modules/tracker/tracker.info.yml...
Checking patch core/modules/update/update.info.yml...
error: while searching for:
name: 'Update Manager'
type: module
description: 'Checks for available updates, and can securely install or update modules and themes via a web interface.'
version: VERSION
package: Core
configure: update.settings

error: patch failed: core/modules/update/update.info.yml:1
error: core/modules/update/update.info.yml: patch does not apply
Checking patch core/modules/user/user.info.yml...
error: while searching for:
name: User
type: module
description: 'Manages the user registration and login system.'
package: Core
version: VERSION
required: true

error: patch failed: core/modules/user/user.info.yml:1
error: core/modules/user/user.info.yml: patch does not apply
Checking patch core/modules/views/views.info.yml...
error: while searching for:
name: Views
type: module
description: 'Create customized lists and queries from your database.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/views/views.info.yml:1
error: core/modules/views/views.info.yml: patch does not apply

paulocs’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
6.91 KB

I didn't have problems to apply the patch. Moving to RTBC as it looks good to me.

catch’s picture

Status: Reviewed & tested by the community » Needs work

The forum module does not provide a forum - it provides the functionality for site builders to create one or several, which other users can then use. The difference is usually visible in a minimal site installation rather then a default.

I don't think this is how a new user would understand the forum module.

The forum module gives you an empty forum (at least in the general sense), because it provides an empty forum index at /forum. You can then add containers and forums within that. Drupal doesn't have a separate word for 'the collection of forums and containers' - it is either 'Forum', 'Forum index' or 'Forums'.

If someone installs phpbb (if that's still going...), then they are installing 'a forum' - even if they then need to create their forum structure within it.

Views + field UI + taxonomy module also provide the functionality for site builders to create one or several forums, but you have to configure things as opposed to just creating content.

This hunk is missing from the patch entirely now, and the patches failed (bot says build successful, but there are thousands of failures), so not sure what's going on with #22. An interdiff would be useful.

adityasingh’s picture

Status: Needs work » Needs review
FileSize
14.81 KB
2.39 KB

Fixed #22 and change description.

sonam.chaturvedi’s picture

Assigned: Unassigned » sonam.chaturvedi
sonam.chaturvedi’s picture

Assigned: sonam.chaturvedi » Unassigned
Status: Needs review » Needs work
FileSize
18.58 KB

As per #3, module description change should also be covered for "RESTful Web Services" module.
The description change for this module is missing in patch#26.

Current description: 'Exposes entities and other resources as RESTful web API'
Expected description (as per patch#4): 'Provides a framework for exposing REST resources.'

Moving to Needs Work

after patch

KapilV’s picture

Assigned: Unassigned » KapilV
KapilV’s picture

Assigned: KapilV » Unassigned
Status: Needs work » Needs review
FileSize
15.42 KB
2.58 KB

Addressed #28

sonam.chaturvedi’s picture

Assigned: Unassigned » sonam.chaturvedi
sonam.chaturvedi’s picture

Assigned: sonam.chaturvedi » Unassigned
Status: Needs review » Reviewed & tested by the community
FileSize
18.01 KB
14.1 KB
16.28 KB
13 KB
18.58 KB
18.78 KB

Verified and tested patch#30. Patch applied successfully and looks good to me.

Testing Steps:
1. Apply patch#30
2. Goto Extend page
3. For below modules, check module description is correct, consistent with "Help text"
"Blocks, CKEditor, Date time, Text editor, Field, Filter, Forum, Help, History, Link, Interface translation, Menu UI, Migrate Drupal, Node, Option, Path, Quick Edit, RDF, Responsive Image, Restful Webservices, Serialization, Shortcut, Testing, Statistics, Syslog, System, Text, Toolbar, Activity tracker, Update Manager, User, Views"

Test Result: Module description of listed modules looks good.

Before Patch
before patch blk
before patch ckedtr
before patch restful

After Patch
after patch blk
after patch ckedtr
after patch restful

Moving to RBTC

catch’s picture

Status: Reviewed & tested by the community » Needs review
  1. +++ b/core/modules/ckeditor/ckeditor.info.yml
    @@ -1,6 +1,6 @@
     name: CKEditor
     type: module
    -description: "WYSIWYG editing for rich text fields using CKEditor."
    +description: 'Provides a visual text editor (WYSIWYG).'
     package: Core
    

    I'm not sure about some of these changes.

    For example in the removed sentence, WYSIWYG is right at the front, meaning you immediately see what it's about, the change puts it at the end.

  2. -description: 'Filters content in preparation for display.'
    +description: 'Allows users to configure text formats that prepare content for display.'
    

    Similarly allows user to configure is mostly redundant here - most modules allow users to configure something - and the information specific to the module is moved to the end of the sentence.

quietone credited Pancho.

quietone credited dawehner.

quietone’s picture

Very nice to see progress on this issue.

Due to my experience with Bug Smash Initiative and looking at old issues I know there are duplicates of this. Let me see if I can find them. Yes, found them, they are:

Closing both as duplicates and adding credit for Pancho and dawehner.

#33. I too thought that some of the changes were odd. But the IS states that this issue is to "Change the descriptions in the modules .info.yml files for the stable modules so that they are correct, consistent, follow the same format, and fit with the wording used in the module's Help text." And that is what the change for CKEditor does. I have not read any of the 'standards' about help texts so I might be missing something important.

And a final note to all, remember to search for duplicates.

Abhijith S’s picture

patch #30 can't be applied.Needs reroll.

Checking patch core/modules/block/block.info.yml...
error: while searching for:
name: Block
type: module
description: 'Controls the visual building blocks a page is constructed with. Blocks are boxes of content rendered into an area, or region, of a web page.'
package: Core
version: VERSION
configure: block.admin_display

error: patch failed: core/modules/block/block.info.yml:1
error: core/modules/block/block.info.yml: patch does not apply
Checking patch core/modules/ckeditor/ckeditor.info.yml...
error: while searching for:
name: CKEditor
type: module
description: "WYSIWYG editing for rich text fields using CKEditor."
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/ckeditor/ckeditor.info.yml:1
error: core/modules/ckeditor/ckeditor.info.yml: patch does not apply
Checking patch core/modules/datetime/datetime.info.yml...
error: while searching for:
name: Datetime
type: module
description: Defines datetime form elements and a datetime field type.
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/datetime/datetime.info.yml:1
error: core/modules/datetime/datetime.info.yml: patch does not apply
Checking patch core/modules/editor/editor.info.yml...
error: while searching for:
name: 'Text Editor'
type: module
description: 'Provides a means to associate text formats with text editor libraries such as WYSIWYGs or toolbars.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/editor/editor.info.yml:1
error: core/modules/editor/editor.info.yml: patch does not apply
Checking patch core/modules/field/field.info.yml...
error: while searching for:
name: Field
type: module
description: 'Field API to add fields to entities like nodes and users.'
package: Core
version: VERSION

error: patch failed: core/modules/field/field.info.yml:1
error: core/modules/field/field.info.yml: patch does not apply
Checking patch core/modules/filter/filter.info.yml...
error: while searching for:
name: Filter
type: module
description: 'Filters content in preparation for display.'
package: Core
version: VERSION
configure: filter.admin_overview

error: patch failed: core/modules/filter/filter.info.yml:1
error: core/modules/filter/filter.info.yml: patch does not apply
Checking patch core/modules/help/help.info.yml...
error: while searching for:
name: Help
type: module
description: 'Manages the display of online help.'
package: Core
version: VERSION

error: patch failed: core/modules/help/help.info.yml:1
error: core/modules/help/help.info.yml: patch does not apply
Checking patch core/modules/history/history.info.yml...
error: while searching for:
name: History
type: module
description: 'Records which user has read which content.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/history/history.info.yml:1
error: core/modules/history/history.info.yml: patch does not apply
Checking patch core/modules/link/link.info.yml...
error: while searching for:
name: Link
type: module
description: 'Provides a simple link field type.'
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/link/link.info.yml:1
error: core/modules/link/link.info.yml: patch does not apply
Checking patch core/modules/locale/locale.info.yml...
error: while searching for:
name: 'Interface Translation'
type: module
description: 'Translates the built-in user interface.'
configure: locale.translate_page
package: Multilingual
version: VERSION

error: patch failed: core/modules/locale/locale.info.yml:1
error: core/modules/locale/locale.info.yml: patch does not apply
Checking patch core/modules/menu_ui/menu_ui.info.yml...
error: while searching for:
name: Menu UI
type: module
description: 'Allows administrators to customize the site navigation menu.'
package: Core
version: VERSION
configure: entity.menu.collection

error: patch failed: core/modules/menu_ui/menu_ui.info.yml:1
error: core/modules/menu_ui/menu_ui.info.yml: patch does not apply
Checking patch core/modules/migrate_drupal/migrate_drupal.info.yml...
error: while searching for:
name: Migrate Drupal
type: module
description: 'Contains migrations from older Drupal versions.'
package: Migration
version: VERSION
dependencies:

error: patch failed: core/modules/migrate_drupal/migrate_drupal.info.yml:1
error: core/modules/migrate_drupal/migrate_drupal.info.yml: patch does not apply
Checking patch core/modules/node/node.info.yml...
error: while searching for:
name: Node
type: module
description: 'Allows content to be submitted to the site and displayed on pages.'
package: Core
version: VERSION
configure: entity.node_type.collection

error: patch failed: core/modules/node/node.info.yml:1
error: core/modules/node/node.info.yml: patch does not apply
Checking patch core/modules/options/options.info.yml...
error: while searching for:
name: Options
type: module
description: 'Defines selection, check box and radio button widgets for text and numeric fields.'
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/options/options.info.yml:1
error: core/modules/options/options.info.yml: patch does not apply
Checking patch core/modules/path/path.info.yml...
error: while searching for:
name: Path
type: module
description: 'Allows users to rename URLs.'
package: Core
version: VERSION
configure: entity.path_alias.collection

error: patch failed: core/modules/path/path.info.yml:1
error: core/modules/path/path.info.yml: patch does not apply
Checking patch core/modules/quickedit/quickedit.info.yml...
error: while searching for:
name: Quick Edit
type: module
description: 'In-place content editing.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/quickedit/quickedit.info.yml:1
error: core/modules/quickedit/quickedit.info.yml: patch does not apply
Checking patch core/modules/rdf/rdf.info.yml...
error: while searching for:
name: RDF
type: module
description: 'Enriches your content with metadata to let other applications (e.g. search engines, aggregators) better understand its relationships and attributes.'
package: Core
version: VERSION

error: patch failed: core/modules/rdf/rdf.info.yml:1
error: core/modules/rdf/rdf.info.yml: patch does not apply
Checking patch core/modules/responsive_image/responsive_image.info.yml...
error: while searching for:
name: Responsive Image
type: module
description: 'Provides an image formatter and breakpoint mappings to output responsive images using the HTML5 picture tag.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/responsive_image/responsive_image.info.yml:1
error: core/modules/responsive_image/responsive_image.info.yml: patch does not apply
Checking patch core/modules/rest/rest.info.yml...
error: while searching for:
name: 'RESTful Web Services'
type: module
description: 'Exposes entities and other resources as RESTful web API'
package: Web services
version: VERSION
dependencies:

error: patch failed: core/modules/rest/rest.info.yml:1
error: core/modules/rest/rest.info.yml: patch does not apply
Checking patch core/modules/serialization/serialization.info.yml...
error: while searching for:
name: Serialization
type: module
description: 'Provides a service for (de)serializing data to/from formats such as JSON and XML.'
package: Web services
version: VERSION

error: patch failed: core/modules/serialization/serialization.info.yml:1
error: core/modules/serialization/serialization.info.yml: patch does not apply
Checking patch core/modules/shortcut/shortcut.info.yml...
error: while searching for:
name: Shortcut
type: module
description: 'Allows users to manage customizable lists of shortcut links.'
package: Core
version: VERSION
configure: entity.shortcut_set.collection

error: patch failed: core/modules/shortcut/shortcut.info.yml:1
error: core/modules/shortcut/shortcut.info.yml: patch does not apply
Checking patch core/modules/statistics/statistics.info.yml...
error: while searching for:
name: Statistics
type: module
description: 'Logs content statistics for your site.'
package: Core
version: VERSION
configure: statistics.settings

error: patch failed: core/modules/statistics/statistics.info.yml:1
error: core/modules/statistics/statistics.info.yml: patch does not apply
Checking patch core/modules/syslog/syslog.info.yml...
error: while searching for:
name: Syslog
type: module
description: 'Logs and records system events to syslog.'
package: Core
version: VERSION
configure: system.logging_settings

error: patch failed: core/modules/syslog/syslog.info.yml:1
error: core/modules/syslog/syslog.info.yml: patch does not apply
Checking patch core/modules/system/system.info.yml...
error: while searching for:
name: System
type: module
description: 'Handles general site configuration for administrators.'
package: Core
version: VERSION
required: true

error: patch failed: core/modules/system/system.info.yml:1
error: core/modules/system/system.info.yml: patch does not apply
Checking patch core/modules/system/tests/src/Functional/Module/PrepareUninstallTest.php...
Checking patch core/modules/text/text.info.yml...
error: while searching for:
name: Text
type: module
description: 'Defines simple text field types.'
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/text/text.info.yml:1
error: core/modules/text/text.info.yml: patch does not apply
Checking patch core/modules/toolbar/toolbar.info.yml...
error: while searching for:
name: Toolbar
type: module
description: 'Provides a toolbar that shows the top-level administration menu links and links from other modules.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/toolbar/toolbar.info.yml:1
error: core/modules/toolbar/toolbar.info.yml: patch does not apply
Checking patch core/modules/tracker/tracker.info.yml...
Checking patch core/modules/update/update.info.yml...
error: while searching for:
name: 'Update Manager'
type: module
description: 'Checks for available updates, and can securely install or update modules and themes via a web interface.'
version: VERSION
package: Core
configure: update.settings

error: patch failed: core/modules/update/update.info.yml:1
error: core/modules/update/update.info.yml: patch does not apply
Checking patch core/modules/user/user.info.yml...
error: while searching for:
name: User
type: module
description: 'Manages the user registration and login system.'
package: Core
version: VERSION
required: true

error: patch failed: core/modules/user/user.info.yml:1
error: core/modules/user/user.info.yml: patch does not apply
Checking patch core/modules/views/views.info.yml...
error: while searching for:
name: Views
type: module
description: 'Create customized lists and queries from your database.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/views/views.info.yml:1
error: core/modules/views/views.info.yml: patch does not apply
Abhijith S’s picture

Rerolled patch #30.please check

quietone’s picture

Abhijith S, When a patch needs a reroll there is no need to show the output from the command, it just makes noise in the issue queue. It will save you time and help keep the focus on the issue. And to help reviewers add an interdiff, or a diff, whichever is appropriate. There are instructions for creating an interdiff.

paulocs’s picture

Status: Needs review » Needs work

I'll provide a patch.

paulocs’s picture

Status: Needs work » Needs review
FileSize
14.98 KB
762 bytes

1 - I didn't have any problem to apply patch #30.
2- I created a new patch that address comment #33 but I'm not sure if it is more clear.

  • 2.1 I revert the change made in ckeditor.info.yml as IMHO it looks good as it is.
  • 2.2 I changed the description of the Filter module.

Cheers, Paulo.

Version: 9.2.x-dev » 9.3.x-dev

Drupal 9.2.0-alpha1 will be released the week of May 3, 2021, which means new developments and disruptive changes should now be targeted for the 9.3.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

BhumikaVarshney’s picture

Version: 9.3.x-dev » 10.0.x-dev
FileSize
19.73 KB
20.68 KB
193.29 KB

Hi @paulocs,
thanks for the patch.
Patch applies cleanly and works as expected.

quietone’s picture

There are two other issues that are modifying the description in info.yml files. Should the work from those be moved into this issue to ensure they also fit with the Help pages?

#2999411: Update the Content Moderation module description
#3033185: Update the description in file.info.yml
Neslee Canil Pinto’s picture

Version: 10.0.x-dev » 9.3.x-dev
Neslee Canil Pinto’s picture

Moved Content Moderation and File modules info.yml description changes here, as per #44

quietone’s picture

@Neslee Canil Pinto, thanks for the interest in this issue. Can you explain why you think it is within scope to add the changes from the other issues here?

I found another issue that tweaks the .info files, #1924996: Fix error and improve on module descriptor text for the Extend page. Again, should those changes move here or is it out of scope?

Neslee Canil Pinto’s picture

@quietone I believe it would be good if we fix all the description-related things under info.yml to be fixed in a single go. what do you think?

quietone’s picture

@Neslee Canil Pinto, I am a bit unsure because of the second paragraph of the Remaining Tasks. And I have not been involved in this issue of the parent so I don't know the full history.

Rinku Jacob 13’s picture

i can't apply patch #46 for drupal 9.3.x-dev. getting error while applying this patch

quietone’s picture

Closed #2999411: Update the Content Moderation module description and #3033185: Update the description in file.info.yml as duplicates. The changes there have already moved over here thanks to Neslee Canil Pinto.

@Rinku Jacob 13 - thank you for looking into this issue. Posting screenshots of your CLI does not advance the issue. So, I've removed the issue credit for that screenshot. In the future, you can get credit for issues by reading the issue to understand its purpose, and posting your review or testing of that purpose. Thank you.

quietone’s picture

Issue summary: View changes
Issue tags: +Needs reroll

Updated the IS

This needs a reroll.

Isabella Rowntree made their first commit to this issue’s fork.

vikashsoni’s picture

Applied patch #30 applied successfully for ref sharing screenshot ..
Thanks for the patch

quietone’s picture

@vikashsoni thank you for looking into this issue. This issue is not ready for screenshots, the previous comment explained that this issue needed a reroll. Once that work is done, and the tests are passing it would be OK to use your time to make screenshots. But using an old patch to make screenshots does not advance the issue. Therefor, credit has been removed per How is credit granted for Drupal core issues.

catch’s picture

volkswagenchick’s picture

Issue tags: +Europe2021

Tagging for Drupal Europe2021. Thanks

kostyashupenko’s picture

Issue tags: -Needs reroll
FileSize
16.11 KB
1.61 KB
eojthebrave’s picture

There is some overlap between this issue and #3223179: Remove references to 'administrators' in module descriptions where the text "administrator" is being removed from module descriptions. Both patches change the description for System and Menu UI. I think we should favor the changes already in this issue since they'll also resolve the issue for those two modules.

That leaves the descriptions for Ban (changed to: "Allows banning visits from specific IP addresses." instead of "Allows administrators to ban visits from specific IP addresses.") and Configuration Manager (changed to: "Allows importing and exporting configuration changes." instead of "Allows administrators to import and export configuration changes.") in the other issue.

It seems to me like we could update the descriptions for Ban and Configuration Manager in this issue and resolve both issue at the same time? Or should that remain a separate change? Either way, once this one is resolved we'll be able to know what to do with the other.

mradcliffe’s picture

Status: Needs review » Needs work

I removed issue credit for Isabella Rowntree as the issue fork contains only unrelated (spam) comments following How is credit granted for Drupal core issues. Thank you, @quietone for the link in the comment above.

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.0-rc1 was released on November 26, 2021, which means new developments and disruptive changes should now be targeted for the 9.4.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

elber’s picture

Assigned: Unassigned » elber
elber’s picture

Assigned: elber » Unassigned
Status: Needs work » Needs review
FileSize
106.26 KB
89.5 KB
hmendes’s picture

Status: Needs review » Needs work

Hey @elber, your patch has a ton of other stuff non related to this issue. Please leave only the chances needed.

elber’s picture

Assigned: Unassigned » elber
elber’s picture

Assigned: elber » Unassigned
Status: Needs work » Needs review
FileSize
17 KB
priyanka.sahni’s picture

Assigned: Unassigned » priyanka.sahni
priyanka.sahni’s picture

Assigned: priyanka.sahni » Unassigned

Verified and tested by applying the patch, patch fails to apply.

Getting below error:
Checking patch core/modules/ban/ban.info.yml...
error: while searching for:
name: Ban
type: module
description: 'Allows administrators to ban visits from specific IP addresses.'
package: Core
version: VERSION
configure: ban.admin_page

error: patch failed: core/modules/ban/ban.info.yml:1
error: core/modules/ban/ban.info.yml: patch does not apply
Checking patch core/modules/block/block.info.yml...
error: while searching for:
name: Block
type: module
description: 'Controls the visual building blocks a page is constructed with. Blocks are boxes of content rendered into an area, or region, of a web page.'
package: Core
version: VERSION
configure: block.admin_display

error: patch failed: core/modules/block/block.info.yml:1
error: core/modules/block/block.info.yml: patch does not apply
Checking patch core/modules/config/config.info.yml...
error: while searching for:
name: 'Configuration Manager'
type: module
description: 'Allows administrators to import and export configuration changes.'
package: Core
version: VERSION
configure: config.sync

error: patch failed: core/modules/config/config.info.yml:1
error: core/modules/config/config.info.yml: patch does not apply
Checking patch core/modules/content_moderation/content_moderation.info.yml...
error: while searching for:
name: 'Content Moderation'
type: module
description: 'Provides moderation states for content.'
version: VERSION
package: Core
configure: entity.workflow.collection

error: patch failed: core/modules/content_moderation/content_moderation.info.yml:1
error: core/modules/content_moderation/content_moderation.info.yml: patch does not apply
Checking patch core/modules/datetime/datetime.info.yml...
error: while searching for:
name: Datetime
type: module
description: Defines datetime form elements and a datetime field type.
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/datetime/datetime.info.yml:1
error: core/modules/datetime/datetime.info.yml: patch does not apply
Checking patch core/modules/editor/editor.info.yml...
error: while searching for:
name: 'Text Editor'
type: module
description: 'Provides a means to associate text formats with text editor libraries such as WYSIWYGs or toolbars.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/editor/editor.info.yml:1
error: core/modules/editor/editor.info.yml: patch does not apply
Checking patch core/modules/field/field.info.yml...
error: while searching for:
name: Field
type: module
description: 'Field API to add fields to entities like nodes and users.'
package: Core
version: VERSION

error: patch failed: core/modules/field/field.info.yml:1
error: core/modules/field/field.info.yml: patch does not apply
Checking patch core/modules/file/file.info.yml...
error: while searching for:
name: File
type: module
description: 'Defines a field type for files.'
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/file/file.info.yml:1
error: core/modules/file/file.info.yml: patch does not apply
Checking patch core/modules/filter/filter.info.yml...
error: while searching for:
name: Filter
type: module
description: 'Filters content in preparation for display.'
package: Core
version: VERSION
configure: filter.admin_overview

error: patch failed: core/modules/filter/filter.info.yml:1
error: core/modules/filter/filter.info.yml: patch does not apply
Checking patch core/modules/help/help.info.yml...
error: while searching for:
name: Help
type: module
description: 'Manages the display of online help.'
package: Core
version: VERSION

error: patch failed: core/modules/help/help.info.yml:1
error: core/modules/help/help.info.yml: patch does not apply
Checking patch core/modules/history/history.info.yml...
error: while searching for:
name: History
type: module
description: 'Records which user has read which content.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/history/history.info.yml:1
error: core/modules/history/history.info.yml: patch does not apply
Checking patch core/modules/link/link.info.yml...
error: while searching for:
name: Link
type: module
description: 'Provides a simple link field type.'
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/link/link.info.yml:1
error: core/modules/link/link.info.yml: patch does not apply
Checking patch core/modules/locale/locale.info.yml...
error: while searching for:
name: 'Interface Translation'
type: module
description: 'Translates the built-in user interface.'
configure: locale.translate_page
package: Multilingual
version: VERSION

error: patch failed: core/modules/locale/locale.info.yml:1
error: core/modules/locale/locale.info.yml: patch does not apply
Checking patch core/modules/menu_ui/menu_ui.info.yml...
error: while searching for:
name: Menu UI
type: module
description: 'Allows administrators to customize the site navigation menu.'
package: Core
version: VERSION
configure: entity.menu.collection

error: patch failed: core/modules/menu_ui/menu_ui.info.yml:1
error: core/modules/menu_ui/menu_ui.info.yml: patch does not apply
Checking patch core/modules/migrate_drupal/migrate_drupal.info.yml...
error: while searching for:
name: Migrate Drupal
type: module
description: 'Contains migrations from older Drupal versions.'
package: Migration
version: VERSION
dependencies:

error: patch failed: core/modules/migrate_drupal/migrate_drupal.info.yml:1
error: core/modules/migrate_drupal/migrate_drupal.info.yml: patch does not apply
Checking patch core/modules/node/node.info.yml...
error: while searching for:
name: Node
type: module
description: 'Allows content to be submitted to the site and displayed on pages.'
package: Core
version: VERSION
configure: entity.node_type.collection

error: patch failed: core/modules/node/node.info.yml:1
error: core/modules/node/node.info.yml: patch does not apply
Checking patch core/modules/options/options.info.yml...
error: while searching for:
name: Options
type: module
description: 'Defines selection, check box and radio button widgets for text and numeric fields.'
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/options/options.info.yml:1
error: core/modules/options/options.info.yml: patch does not apply
Checking patch core/modules/path/path.info.yml...
error: while searching for:
name: Path
type: module
description: 'Allows users to rename URLs.'
package: Core
version: VERSION
configure: entity.path_alias.collection

error: patch failed: core/modules/path/path.info.yml:1
error: core/modules/path/path.info.yml: patch does not apply
Checking patch core/modules/quickedit/quickedit.info.yml...
error: while searching for:
name: Quick Edit
type: module
description: 'In-place content editing.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/quickedit/quickedit.info.yml:1
error: core/modules/quickedit/quickedit.info.yml: patch does not apply
Checking patch core/modules/rdf/rdf.info.yml...
error: while searching for:
name: RDF
type: module
description: 'Enriches your content with metadata to let other applications (e.g. search engines, aggregators) better understand its relationships and attributes.'
package: Core
version: VERSION

error: patch failed: core/modules/rdf/rdf.info.yml:1
error: core/modules/rdf/rdf.info.yml: patch does not apply
Checking patch core/modules/responsive_image/responsive_image.info.yml...
error: while searching for:
name: Responsive Image
type: module
description: 'Provides an image formatter and breakpoint mappings to output responsive images using the HTML5 picture tag.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/responsive_image/responsive_image.info.yml:1
error: core/modules/responsive_image/responsive_image.info.yml: patch does not apply
Checking patch core/modules/rest/rest.info.yml...
error: while searching for:
name: 'RESTful Web Services'
type: module
description: 'Exposes entities and other resources as RESTful web API'
package: Web services
version: VERSION
dependencies:

error: patch failed: core/modules/rest/rest.info.yml:1
error: core/modules/rest/rest.info.yml: patch does not apply
Checking patch core/modules/serialization/serialization.info.yml...
error: while searching for:
name: Serialization
type: module
description: 'Provides a service for (de)serializing data to/from formats such as JSON and XML.'
package: Web services
version: VERSION

error: patch failed: core/modules/serialization/serialization.info.yml:1
error: core/modules/serialization/serialization.info.yml: patch does not apply
Checking patch core/modules/shortcut/shortcut.info.yml...
error: while searching for:
name: Shortcut
type: module
description: 'Allows users to manage customizable lists of shortcut links.'
package: Core
version: VERSION
configure: entity.shortcut_set.collection

error: patch failed: core/modules/shortcut/shortcut.info.yml:1
error: core/modules/shortcut/shortcut.info.yml: patch does not apply
Checking patch core/modules/statistics/statistics.info.yml...
error: while searching for:
name: Statistics
type: module
description: 'Logs content statistics for your site.'
package: Core
version: VERSION
configure: statistics.settings

error: patch failed: core/modules/statistics/statistics.info.yml:1
error: core/modules/statistics/statistics.info.yml: patch does not apply
Checking patch core/modules/syslog/syslog.info.yml...
error: while searching for:
name: Syslog
type: module
description: 'Logs and records system events to syslog.'
package: Core
version: VERSION
configure: system.logging_settings

error: patch failed: core/modules/syslog/syslog.info.yml:1
error: core/modules/syslog/syslog.info.yml: patch does not apply
Checking patch core/modules/system/system.info.yml...
error: while searching for:
name: System
type: module
description: 'Handles general site configuration for administrators.'
package: Core
version: VERSION
required: true

error: patch failed: core/modules/system/system.info.yml:1
error: core/modules/system/system.info.yml: patch does not apply
Checking patch core/modules/system/tests/src/Functional/Module/PrepareUninstallTest.php...
Checking patch core/modules/text/text.info.yml...
error: while searching for:
name: Text
type: module
description: 'Defines simple text field types.'
package: Field types
version: VERSION
dependencies:

error: patch failed: core/modules/text/text.info.yml:1
error: core/modules/text/text.info.yml: patch does not apply
Checking patch core/modules/toolbar/toolbar.info.yml...
error: while searching for:
name: Toolbar
type: module
description: 'Provides a toolbar that shows the top-level administration menu links and links from other modules.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/toolbar/toolbar.info.yml:1
error: core/modules/toolbar/toolbar.info.yml: patch does not apply
Checking patch core/modules/tracker/tracker.info.yml...
Checking patch core/modules/update/update.info.yml...
error: while searching for:
name: 'Update Manager'
type: module
description: 'Checks for available updates, and can securely install or update modules and themes via a web interface.'
version: VERSION
package: Core
configure: update.settings

error: patch failed: core/modules/update/update.info.yml:1
error: core/modules/update/update.info.yml: patch does not apply
Checking patch core/modules/user/user.info.yml...
error: while searching for:
name: User
type: module
description: 'Manages the user registration and login system.'
package: Core
version: VERSION
required: true

error: patch failed: core/modules/user/user.info.yml:1
error: core/modules/user/user.info.yml: patch does not apply
Checking patch core/modules/views/views.info.yml...
error: while searching for:
name: Views
type: module
description: 'Create customized lists and queries from your database.'
package: Core
version: VERSION
dependencies:

error: patch failed: core/modules/views/views.info.yml:1
error: core/modules/views/views.info.yml: patch does not apply

Version: 9.4.x-dev » 9.5.x-dev

Drupal 9.4.0-alpha1 was released on May 6, 2022, which means new developments and disruptive changes should now be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Rinku Jacob 13’s picture

FileSize
83.99 KB
95.72 KB

I have successfully applied the patch#68 for drupal version 9.4.x-dev. Adding screenshots for the reference

aarti zikre’s picture

Status: Needs review » Reviewed & tested by the community
quietone’s picture

Status: Reviewed & tested by the community » Needs work

@priyanka.sahni, if a patch does not apply locally just comment that it did not apply and the version of Drupal core you were using. Adding the terminal output is noise and doesn't advance the issue.

@Rinku Jacob 13, screenshots are not needed for this issue. It would be difficult to show all the strings and it would be better to just to manual testing and report back. And your screenshot just one of the many instances of changed text in the patch. Therefor, credit has been removed per How is credit granted for Drupal core issues.

@aarti zikre, thanks for the confidence in this patch. However, you comment should state what you did to confirm that this patch is ready to be committed to core.

Setting back to NW for manual testing.

Munavijayalakshmi’s picture

Assigned: Unassigned » Munavijayalakshmi
Munavijayalakshmi’s picture

Status: Needs work » Needs review
FileSize
16.61 KB

Re-rolled #68 patch.

Munavijayalakshmi’s picture

Assigned: Munavijayalakshmi » Unassigned
Munavijayalakshmi’s picture

ranjith_kumar_k_u’s picture

Status: Needs review » Needs work

The last submitted patch, 79: 3060616-79.patch, failed testing. View results

quietone’s picture

Issue summary: View changes
FileSize
7.11 KB
1.39 KB
17 KB

@Munavijayalakshmi, Thanks for looking into this issue. The patch did not need a reroll. The patch still applies to 9.5. The previous comment, #74, states that the work needed here is manual testing. I am surprised that applying the patch locally didn't show that a reroll was not needed. Oh, looking at the patch, you were working on Drupal 8, which is no longer supported. And for future reference, when rerolling always provide an interdiff or a diff file and comment on what you did. There are instructions for creating an interdiff .

After making a few diffs and interdiffs, I see that the patch in #76 has introduced changes and errors. The strings for menu and migrate_drupal are 'Provides functionality to output responsive images using the HTML5 picture tag.'. which is incorrect. I'll upload the diffs I made. The patch in #79 is a reroll of the patch from #78, continuing the errors.

The patch to use here is the patch in #68. I will upload it again to help avoid confusion. I am not running tests, we can do that when this is ready for RTBC.

I have updated the remaining tasks in the Issue Summary.

I hope all that helps. This is so close ....

quietone’s picture

Status: Needs work » Needs review

Meant to set to Needs review to get eyes on all the changes.

Version: 9.5.x-dev » 10.1.x-dev

Drupal 9.5.0-beta2 and Drupal 10.0.0-beta2 were released on September 29, 2022, which means new developments and disruptive changes should now be targeted for the 10.1.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

nod_’s picture

Status: Needs review » Reviewed & tested by the community

By now I think the CKEditor 5 module is the only one that's not consisitent with the rest.

I'm not a native speaker so I can't say how good it is, for me the description reads fine. It's better than it used to be so RTBC.

longwave’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Patch needs reroll for 10.1.x. Now that the 9.5.0 and 10.0.0 release candidates are out I don't think we can backport this as it contains translatable string changes.

elber’s picture

Assigned: Unassigned » elber

I will do a reroll

elber’s picture

Assigned: elber » Unassigned
Status: Needs work » Needs review
FileSize
18.65 KB

Hi please revise.

nod_’s picture

Thanks! unfortunately the patch contains some unrelated changes and files: .gitignore, .lando.yml, 3322094-5.patch. they're not related to this issue so they shouldn't appear in a patch here.

Rerolled and tried to update the CKE5 text, needs review :)

elber’s picture

Sorry @nod I forgot to see and delete it, but I will review your reroll

nod_’s picture

no worries, there are days I need a fair bit of tries before uploading a complete patch too :)

elber’s picture

Status: Needs review » Reviewed & tested by the community

Hi I revised.

I was able to apply the reroll in Drupal 10.1.x version
descriptions are fine.

For me issue was resolved.

quietone’s picture

Status: Reviewed & tested by the community » Needs work

I read The help text standards, noting "The description starts with a verb and should be short and concise.'. I then applied the patch and read the changes on a Drupal 10.1.x, standard install. I notice the following, which do not meet that standard.

These do not start with a verb.

  • Filter: Filter text content formats in preparation for display. - change TBD
  • MySQLl: Database driver for MySQL. - change to 'Provides the MySQL database driver.'
  • PostreSQL: Database driver for PostgreSQL. - change to 'Provides the PostgreSQL database driver.'
  • SQLite: Database driver for SQLite. - change to 'Provides the SQLite database driver.'

These are inconsistent.

  • File: Defines a "managed_file" Form API element and a "file" for Field module. - This is inconsistent with the other modules that provide fields. Those modules use the phrase 'provides a field type'. Perhaps this should be 'Provides a field type for files and defines a "managed_file" Form API element'.
  • HTTP Basic Authentication: Supplies an HTTP Basic authentication provider - This is only use of 'Supplies' on the page. Let's change it to 'Provides'

And finally,
Migrate: Handles migrations. - This is insufficient. Something along the lines of "Provides a framework for migrating data to Drupal."
CKeditor 5: Provides a rich text editor for fields using CKEditor 5. - This is self referring. Maybe, 'Provides the CKEditor 5 rich text editor".

That is 8 lines to change. I checked the IS for scope and it does start out stating that this is a subset of the descriptions. Looking at the other issue this appears to be for fixing the ones not corrected there. So, I think this is in scope.

Setting to NW.

nod_’s picture

Status: Needs work » Needs review
FileSize
18.56 KB
2.74 KB

Thanks :) Kinda cheated for the filter description but that's the limit of my skills with words.

Bhanu951’s picture

Issue tags: -Needs reroll
longwave’s picture

Status: Needs review » Reviewed & tested by the community
+++ b/core/modules/filter/filter.info.yml
@@ -1,6 +1,6 @@
+description: 'Filters text content formats in preparation for display.'

We could possibly drop the word "formats" here, but this is the only nit I could find. Marking RTBC as this all looks good again now.

quietone’s picture

To other committers. I'd like to finish this one and commit in the next 24 hours. Thanks.

quietone’s picture

Or maybe not. I am having trouble with my signing key.

  • quietone committed d8e5990 on 10.1.x
    Issue #3060616 by elber, nod_, quietone, paulocs, Neslee Canil Pinto,...
quietone’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: -Needs issue summary update, -Novice

It is very nice to have consistent language now on the modules page. Thanks!

This is a string change, see #9, so only eligible for 10.1.x.

Thanks everyone!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.