CommentFileSizeAuthor
#2 t_2820742_1.patch13.99 KBnehakhadke
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nehakhadke created an issue. See original summary.

nehakhadke’s picture

nehakhadke CreditAttribution: nehakhadke commented
FileSize
t_2820742_1.patch13.99 KB
naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Status: Active » Needs review
Issue tags: +Novice

Thanks!

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

Tested module with patch and its working.

  • neffets committed 5c6176d on 8.x-1.x authored by nehakhadke 
    Issue #2820742 by nehakhadke: $this->t() should be used instead of t()...
neffets’s picture

neffets CreditAttribution: neffets commented
Assigned: Unassigned » neffets
Status: Reviewed & tested by the community » Fixed

patch added. Thanx to nehakhadke

neffets’s picture

neffets CreditAttribution: neffets commented

Had to change some line of the path, because it caused errors:

Error: Using $this when not in object context in Drupal\iframe\Plugin\Field\FieldType\IframeItem::propertyDefinitions() (line 49 of modules/iframe/src/Plugin/Field/FieldType/IframeItem.php).

In static functions it not usable

Status: Fixed » Closed (fixed)

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

nehakhadke’s picture

nehakhadke CreditAttribution: nehakhadke at Invezza Technologies commented