StripTags and Trim shouldn't fail if the argument is empty

Created on 4 August 2021, over 3 years ago
Updated 19 May 2023, over 1 year ago

Problem/Motivation

When using a plugin it is common for values to be empty, because it's possible that not every record has a non-empty value. When this happens the Trim::tamper() or StripTags::tamper() methods, an exception is thrown even though the value should be acceptable.

Steps to reproduce

Use Feeds to parse a CSV. Configure Feeds Tamper to use the Trim plugin on all columns. Empty values are passed to Trim::tamper() as NULL.

Proposed resolution

Ignore NULL or empty ("") strings.

Remaining tasks

Provide a patch.

User interface changes

TBD

API changes

TBD

Data model changes

TBD

🐛 Bug report
Status

Fixed

Version

1.0

Component

Code

Created by

🇺🇸United States DamienMcKenna NH, USA

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024