Add call parent submitForm in SettingsForm.

CommentFileSizeAuthor
#2 parent_submitForm-3068334-2.patch457 bytesSergiu Stici
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sergiu Stici created an issue. See original summary.

Sergiu Stici’s picture

Here is the patch, please review.

Sergiu Stici’s picture

Status: Active » Needs review
BarisW’s picture

Status: Needs review » Closed (works as designed)

I have no idea why this should be done. Please take a bit more time to elaborate on why an change is needed.

Sergiu Stici’s picture

Status: Closed (works as designed) » Needs review

It's executing a method with the same name on its parent class. This is standard fare in any language when you're dealing with inheritance, when functionality from the base class is still (or may be in the future) required.

Have a read up on Dependency Injection for a Form if this concept is unfamiliar to you.

BarisW’s picture

Status: Needs review » Reviewed & tested by the community

Alright, thanks for the info!

  • BarisW committed 3fbfefa on 8.x-1.x authored by Sergiu Stici
    Issue #3068334 by Sergiu Stici, BarisW: Add call parent submitForm
    
BarisW’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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