This is a really small error (hence marking it as private). One of the references in docblock for MigrateSourcePluginManager is broken. This results in a missing link in documentation page as well. It is a quickfix.

It's really simple -- \Drupal\migrate\Plugin\source\SourcePluginBase should be \Drupal\migrate\Plugin\migrate\source\SourcePluginBase

I am not sure if there is an automated way to catch all these. If there is, we should do that instead. I'll go ahead and document this anyway.

CommentFileSizeAuthor
#2 2895180.patch633 byteshussainweb

Comments

hussainweb created an issue. See original summary.

hussainweb’s picture

hussainweb
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new 2895180.patch633 bytes

Simple patch

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Assigned: Unassigned » quietone

Assigning to vicki to review this week.

quietone’s picture

quietone commented
Status: Needs review » Reviewed & tested by the community

@hussainweb, thanks for fixing our docs!

quietone’s picture

quietone commented
Assigned: quietone » Unassigned

Version: 8.4.x-dev » 8.5.x-dev

Drupal 8.4.0-alpha1 will be released the week of July 31, 2017, which means new developments and disruptive changes should now be targeted against the 8.5.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

catch’s picture

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

Committed/pushed to 8.5.x and cherry-picked to 8.4.x. Thanks!

  • catch committed a06e758 on 8.5.x 
    Issue #2895180 by hussainweb: Fix documentation in...

  • catch committed c0653f1 on 8.4.x 
    Issue #2895180 by hussainweb: Fix documentation in...

Status: Fixed » Closed (fixed)

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