I'm dealing with data through feeds where fields are base64 encoded. I figured it would be a nice addition to the default plugins and it's an easy implementation. Needs review, works good locally.

CommentFileSizeAuthor
#8 feeds_tamper-add_encoder_decoder-2224595-8.patch2.97 KBthePanz
#5 feeds_tamper-add_encoder_decoder-2224595-5.patch2.97 KBthePanz
#3 feeds_tamper-add_encoder_decoder-2224595-3.patch3.05 KBthePanz
feeds_tamper-base64.patch1.19 KBblazindrop
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

twistor’s picture

twistor CreditAttribution: twistor commented
Status: Needs review » Needs work
Issue tags: +Needs tests 
+++ b/docroot/sites/all/modules/contrib/feeds_tamper/plugins/base64.inc
@@ -0,0 +1,36 @@
+ * Convert text to upper, lower, or title case.

hmmm... :)

This looks good. Just needs a couple tests.

kamescg’s picture

kamescg CreditAttribution: kamescg commented

EDIT - Irrelevant comment

thePanz’s picture

thePanz CreditAttribution: thePanz at Liip for FREITAG lab. AG commented
Title: Base64 encoding/decoding plugin » Encoding/decoding plugin
Status: Needs work » Needs review
FileSize
feeds_tamper-add_encoder_decoder-2224595-3.patch3.05 KB

This patch adds a Plugin with set of encoders/decoders; the following modes are available:

- serializa/unserialize from PHP
- base64_encode/base64_decode
- json_encode/json_decode

Preliminary tests have been added, too.

Status: Needs review » Needs work

The last submitted patch, 3: feeds_tamper-add_encoder_decoder-2224595-3.patch, failed testing.

thePanz’s picture

thePanz CreditAttribution: thePanz at Liip for FREITAG lab. AG commented
FileSize
feeds_tamper-add_encoder_decoder-2224595-5.patch2.97 KB

Removed unused _validate callback.

thePanz’s picture

thePanz CreditAttribution: thePanz at Liip for FREITAG lab. AG commented
Assigned: blazindrop » Unassigned
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 5: feeds_tamper-add_encoder_decoder-2224595-5.patch, failed testing.

thePanz’s picture

thePanz CreditAttribution: thePanz at Liip for FREITAG lab. AG commented
Status: Needs work » Needs review
FileSize
feeds_tamper-add_encoder_decoder-2224595-8.patch2.97 KB

Updated patch

sher1’s picture

sher1 CreditAttribution: sher1 as a volunteer commented

I patched my feeds_tamper with this patch (great idea by the way) but the new encode/decode options aren't showing up in the plugin list. The file is there in the plugins directory but I can't use it.

thePanz’s picture

thePanz CreditAttribution: thePanz at Liip for FREITAG lab. AG commented

@sher1: did you already try to clear the Drupal cache and/or the Drupal registry?

sher1’s picture

sher1 CreditAttribution: sher1 as a volunteer commented

The key is to check to see which copy of feeds_tamper you are using. I had patched the one in sites/all/modules and not the local copy that my site was using. It shows up just fine. Sorry for the false alarm.

thePanz’s picture

thePanz CreditAttribution: thePanz at Liip for FREITAG lab. AG commented

No problem @sher1! :) Feel free to update the status of this issue to "Review & Tested by the community" if it is working for you, thanks!

Watergate’s picture

Watergate CreditAttribution: Watergate at Sicse commented
Status: Needs review » Reviewed & tested by the community

Thanks for the patch and I can confirm that the base64 decoding works. I've looked at the code and this seems fine to me. Marking this as RTBC.

  • MegaChriz committed 2058693 on 7.x-1.x authored by thePanz 
    Issue #2224595 by thePanz, blazindrop, Watergate, MegaChriz: Added...
MegaChriz’s picture

MegaChriz CreditAttribution: MegaChriz as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Looks good to me too. I made a few changes in the code comments on commit.

Status: Fixed » Closed (fixed)

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