Problem/Motivation

Currently it's pretty hard to access third party settings from the field formatter in a custom RiftBuilder plugin, without overwriting large portions of the module. The PictureConfig DTO (used for example in the isApplicable() method) has no support for other values than the hardcoded ones.

Proposed resolution

Add third party settings to the PictureConfig DTO

Remaining tasks

Implement the feature

Issue fork rift-3533700

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

sch4lly created an issue. See original summary.

sch4lly’s picture

Status: Active » Needs review
d34dman’s picture

@sch4lly,

This is indeed a good improvement, Goes along the lines of what I had in mind for 2.x release. Make it more easy to extend via other custom/contrib modules.

I tested this, and it doesn't seems to break anything. However when i run the existing site test on this change, it is failing.

This also points out we need to add some linters into the project.

Is it ok if I put this on hold until I figure out why the tests are failing?

d34dman’s picture

  • d34dman committed f02e5f89 on 2.x authored by sch4lly
    Issue #3533700 by sch4lly, d34dman: Add support for third party settings...
d34dman’s picture

Version: 1.x-dev » 2.x-dev
Status: Needs review » Fixed

Made some minor adjustments and got this in.

- Renamed the new method to follow naming patterns for other methods.
- Added an empty "third_part_settings" to default config so that usage without field formatter wont' lead to broken code.

Status: Fixed » Closed (fixed)

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

d34dman’s picture

Parent issue: » #3540394: Roadmap for 2.0.0