Problem/Motivation

Typo in API documentation at https://api.drupal.org/api/drupal/core%21includes%21form.inc/group/batch...

Proposed resolution

core/includes/form.inc

@@ -670,7 +670,7 @@ function template_preprocess_form_element_label(&$variables): void {
 *   foreach ($results as $result) {
- *     $items[] = t('Loaded node %title.', ['%title' => $result)];
+ *     $items[] = t('Loaded node %title.', ['%title' => $result]);
 *   }

@@ -702,7 +702,7 @@ function template_preprocess_form_element_label(&$variables): void {
 *     @code
 *     [
- *       ['callback_batch_operation_1', [$arg1)],
+ *       ['callback_batch_operation_1', [$arg1]],
 *     ]
 *     @endcode

Issue fork drupal-3510498

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:

About issue forks

Comments

frondeau created an issue. See original summary.

frondeau’s picture

frondeau commented
Issue summary: View changes
frondeau’s picture

frondeau commented
Issue summary: View changes
cilefen’s picture

cilefen commented
Category: Task » Bug report
Issue tags: +Novice
nexusnovaz’s picture

nexusnovaz commented
Assigned: Unassigned » nexusnovaz

Ill take a quick look at this. Should be quick

nexusnovaz opened merge request !11339

nexusnovaz’s picture

nexusnovaz commented
Assigned: nexusnovaz » Unassigned
Status: Active » Needs review

I believe MR !11339 is ready for review. I also found one more which I believe was also a mistake.

ankitv18’s picture

ankitv18 commented
Status: Needs review » Needs work

Minor change required on MR.

danrod made their first commit to this issue’s fork.

danrod’s picture

danrod
Location Ottawa
commented
Status: Needs work » Needs review
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
Status: Needs review » Reviewed & tested by the community

This looks like a good documentation update, it makes sense to fix this. The scope here is quite small, which makes sense for a novice issue.

sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented
Title: parentheses mistake » Parentheses typo in documentation
Issue summary: View changes

+1 for RTBC.

  • nod_ committed 427d7455 on 11.x 
    Issue #3510498 by nexusnovaz, danrod, frondeau, ankitv18, borisson_: Fix...

nod_ closed merge request !11339

nod_’s picture

nod_
Primary language French
Location Lille
commented
Title: Parentheses typo in documentation » Fix parentheses mistake in sample code documentation
Status: Reviewed & tested by the community » Fixed

Committed 427d745 and pushed to 11.x. Thanks!

Status: Fixed » Closed (fixed)

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