There are a few typos in the core/modules/aggregator/src/Plugin/migrate/source/AggregatorFeed.php file, for names of fields.

CommentFileSizeAuthor
#6 typo-aggregator-2638014-6.patch1013 bytesddrozdik
#2 typo-aggregator-2638014.patch1014 bytesddrozdik
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ddrozdik created an issue. See original summary.

ddrozdik’s picture

ddrozdik CreditAttribution: ddrozdik as a volunteer commented
Assigned: ddrozdik » Unassigned
Status: Active » Needs review
FileSize
typo-aggregator-2638014.patch1014 bytes
cilefen’s picture

cilefen CreditAttribution: cilefen commented
Version: 8.0.x-dev » 8.1.x-dev
Priority: Minor » Normal

String changes have to go in the next minor release.

cilefen’s picture

cilefen CreditAttribution: cilefen commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch as a volunteer commented
Status: Reviewed & tested by the community » Needs work

While we're changing it, let's use double quotes for the string containing the single quote so that translators don't have to deal with escaping.

ddrozdik’s picture

ddrozdik CreditAttribution: ddrozdik as a volunteer commented
Status: Needs work » Needs review
FileSize
typo-aggregator-2638014-6.patch1013 bytes

Done.

cilefen’s picture

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

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

Committed/pushed to 8.1.x, thanks!

  • catch committed 28b1de1 on 8.1.x 
    Issue #2638014 by ddrozdik: Typo in the aggregator module

Status: Fixed » Closed (fixed)

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