Reading the documentation I found a typo in core.api.php. The text reads:

* - Defining the default configuration for an extension (module, theme, or
* profile), which is imported to the active storage when the extension if
* enabled.

Correctly it should be is instead of if:

* - Defining the default configuration for an extension (module, theme, or
* profile), which is imported to the active storage when the extension is
* enabled.

CommentFileSizeAuthor
#8 typo_extension_if_corrected-2621216-8.patch703 bytesnirvana_u
#7 typo_extension_if_corrected-2621216-6.patch1.21 KBnirvana_u
#7 typo_extension_if_corrected-2621216-6.patch1.21 KBnirvana_u
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

balagan created an issue. See original summary.

zsofi.major’s picture

zsofi.major CreditAttribution: zsofi.major at Cheppers commented
Assigned: Unassigned » zsofi.major
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Version: 8.0.0-rc4 » 8.0.x-dev
balagan’s picture

balagan CreditAttribution: balagan as a volunteer commented

Yeah, maybe it is in -dev, I just saw it online.

nirvana_u’s picture

nirvana_u CreditAttribution: nirvana_u as a volunteer and at Faichi Solutions Pvt Ltd commented
Assigned: zsofi.major » nirvana_u
monika.kadam1206’s picture

monika.kadam1206 CreditAttribution: monika.kadam1206 as a volunteer and at Faichi Solutions Pvt Ltd commented
Assigned: nirvana_u » monika.kadam1206
nirvana_u’s picture

nirvana_u CreditAttribution: nirvana_u as a volunteer and at Faichi Solutions Pvt Ltd commented
Assigned: monika.kadam1206 » Unassigned
Status: Active » Needs review
FileSize
typo_extension_if_corrected-2621216-6.patch1.21 KB
typo_extension_if_corrected-2621216-6.patch1.21 KB

Hi @balagan,

I have created a patch for the same. Please review and merge if it seems okay.

Thanks!!
nirvana_u

nirvana_u’s picture

nirvana_u CreditAttribution: nirvana_u as a volunteer and at Faichi Solutions Pvt Ltd commented
FileSize
typo_extension_if_corrected-2621216-8.patch703 bytes

Hi @balagan,

I have created a patch for the same. Please review and merge if it seems okay.
Please ignore the above patch and go for #8.

Thanks!!
nirvana_u

anup.singh’s picture

anup.singh CreditAttribution: anup.singh as a volunteer and at Faichi Solutions Pvt Ltd commented
Status: Needs review » Reviewed & tested by the community

Hi nirvana_u,

Patch is working fine in my drupal installation.

Thanks
anup.singh

The last submitted patch, 7: typo_extension_if_corrected-2621216-6.patch, failed testing.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.1.x and cherry-picked to 8.0.x. Thanks!

  • catch committed 5bff82c on 8.1.x 
    Issue #2621216 by nirvana_u: Typo: "extension if" instead of "extension...

  • catch committed 157cbc1 on 
    Issue #2621216 by nirvana_u: Typo: "extension if" instead of "extension...
nirvana_u’s picture

nirvana_u CreditAttribution: nirvana_u as a volunteer and at Faichi Solutions Pvt Ltd commented

Hi catch,

Thanks a lot for committing the patch for drupal core.

Thanks!!
nirvana_u

Status: Fixed » Closed (fixed)

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