Transform: sed (aka replace)

[andylolz]

Well… we could at least partially implement this.

[rufuspollock]

Basic find and replace would be really useful. Would not go for full sed :-)

[rufuspollock]

@andylolz i'm trying to use find and replace and keep getting errors

When I try clicking on the example on the page i get an error

http://datapipes.okfnlabs.org/replace/

Here's my link

http://datapipes.okfnlabs.org/csv/replace%20,%20$r/html?r=&url=http://www.cps.gov.uk/data/case_outcomes/february_2012/principal_offence_cat_nat_sum_february_2012.csv

Also how do i replace with nothing - surely second argument should be optional (= empty string).

[andylolz]

Also how do i replace with nothing - surely second argument should be optional (= empty string).

V good point! That seems like exactly the solution to the problem you’re having here.

[rufuspollock]

@andylolz just realized that part of this was that first error had blow the whole system so ignore issue with example - and cf #97

[rufuspollock]

Got this working - no real bug but some usability plus #97.

Another example for the examples file:

http://datapipes.okfnlabs.org/csv/replace%20-c3%20,%20%24r/html?r=&url=http://www.cps.gov.uk/data/case_outcomes/february_2012/principal_offence_cat_nat_sum_february_2012.csv

This removes the comma in the 4th column numbers. Note we really need to fix #97.

[andylolz]

Usability issue should now be fixed (b3bcdad). So based on your earlier suggestion, your example becomes:

http://datapipes.okfnlabs.org/csv/replace%20-c3%20,/html?url=http://www.cps.gov.uk/data/case_outcomes/february_2012/principal_offence_cat_nat_sum_february_2012.csv

Agree that #97 is pressing. I’ve bumped up the priority. UPDATE: #97 should be fixed now.

[rufuspollock]

@andylolz do we mark this as fixed given we have basic replace?

[andylolz]

Sounds good – although we still need some unit tests for this one.