There's a minor spelling mistake in Feeds module.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Munavijayalakshmi created an issue. See original summary.

Munavijayalakshmi’s picture

Assigned: Munavijayalakshmi » Unassigned
Status: Active » Needs review
FileSize
10.69 KB

MegaChriz’s picture

Status: Needs review » Fixed

Thanks for the patch. Committed #2 with a few changes:

  1. improper => type (should have been 'importer' in an earlier version of Feeds)
  2. plugings => plugins
  3. Left out the following:
    +++ b/src/FeedForm.php
    @@ -4,7 +4,7 @@ namespace Drupal\feeds;
    -use Drupal\Core\Entity\EntityManagerInterface;
    +use Drupal\Core\Entity\EntityTypeManagerInterface;
    
    @@ -28,7 +28,7 @@ class FeedForm extends ContentEntityForm {
    -   * @param \Drupal\Core\Entity\EntityManagerInterface $entity_manager
    +   * @param \Drupal\Core\Entity\EntityTypeManagerInterface $entity_type_manager
    
    @@ -37,8 +37,8 @@ class FeedForm extends ContentEntityForm {
    -  public function __construct(EntityManagerInterface $entity_manager, PluginFormFactory $factory, EntityTypeBundleInfoInterface $entity_type_bundle_info = NULL, TimeInterface $time = NULL) {
    -    parent::__construct($entity_manager, $entity_type_bundle_info, $time);
    +  public function __construct(EntityTypeManagerInterface $entity_type_manager, PluginFormFactory $factory, EntityTypeBundleInfoInterface $entity_type_bundle_info = NULL, TimeInterface $time = NULL) {
    +    parent::__construct($entity_type_manager, $entity_type_bundle_info, $time);
    
    @@ -55,7 +55,7 @@ class FeedForm extends ContentEntityForm {
    -      $container->get('entity.manager'),
    +      $container->get('entity_type.manager'),
    

Status: Fixed » Closed (fixed)

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