Closed (fixed)
Project:
Responsive Image Formatter Tools (RIFT)
Version:
2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
2 Jul 2025 at 13:49 UTC
Updated:
8 Aug 2025 at 08:10 UTC
Jump to comment: Most recent
Comments
Comment #5
sch4lly commentedComment #6
d34dman commented@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?
Comment #7
d34dman commentedComment #9
d34dman commentedMade 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.
Comment #11
d34dman commented