Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
When creating a CSV parser plugin to override the default FeedsCSVParse, the feeds_tamper checks for the class name (hardcoing the class "FeedsCSVParser") instead of checking if the parser is of instance of FeedsCSVParser. Getting the class name wrong creates a wrong value for the $is_csv variable. Using "instanceof" instead of get_class() can avoid this issue.
Comment | File | Size | Author |
---|---|---|---|
#1 | use_instancof_instead_of_get_class-2490120-1.patch | 1 KB | ranroz |
Comments
Comment #1
ranroz CreditAttribution: ranroz commentedComment #2
twistor CreditAttribution: twistor commentedComment #4
twistor CreditAttribution: twistor commentedGood call! Old code :(