--- a/fastx_clipper.xml	Wed Nov 11 12:38:51 2015 -0500
+++ b/fastx_clipper.xml	Tue May 08 13:27:25 2018 -0400
@@ -1,23 +1,26 @@
-<tool id="cshl_fastx_clipper" name="Clip" version="1.0.1">
+<tool id="cshl_fastx_clipper" name="Clip" version="1.0.2">
     <description>adapter sequences</description>
-    <requirements>
-        <requirement type="package" version="0.0.13">fastx_toolkit</requirement>
-    </requirements>
-    <command>
-<![CDATA[
-zcat -f < '$input' | fastx_clipper -l $minlength -a '$clip_source.clip_sequence' -d $keepdelta -o '$output' -v $KEEP_N $DISCARD_OPTIONS
-#if $input.ext == "fastqsanger":
-   -Q 33
-#end if
-]]>
-    </command>
+    <macros>
+        <import>macros.xml</import>
+    </macros>
+    <expand macro="requirements" />
+    <command detect_errors="exit_code"><![CDATA[
+@CATS@ fastx_clipper
+-l $minlength
+-a '$clip_source.clip_sequence'
+-d $keepdelta
+-o '$output'
+-v
+$KEEP_N
+$DISCARD_OPTIONS
+@FQQUAL@
+    ]]></command>
 
     <inputs>
-        <param format="fasta,fastqsanger,fastqsolexa,fastqillumina" name="input" type="data" label="Library to clip" />
+        <expand macro="fastx_input" />
 
-        <param name="minlength" type="integer" value="15">
-            <label>Minimum sequence length (after clipping, sequences shorter than this length will be discarded)</label>
-        </param>
+        <param name="minlength" type="integer" value="15"
+            label="Minimum sequence length (after clipping, sequences shorter than this length will be discarded" />
 
         <conditional name="clip_source">
             <param name="clip_source_list" type="select" label="Source">
@@ -39,10 +42,9 @@
             </when>
         </conditional>
 
-        <param name="keepdelta" type="integer" value="0">
-            <label>enter non-zero value to keep the adapter sequence and x bases that follow it</label>
-            <help>use this for hairpin barcoding. keep at 0 unless you know what you're doing.</help>
-        </param>
+        <param name="keepdelta" type="integer" value="0"
+            label="Enter a non-zero value to keep the adapter sequence and x bases that follow it"
+            help="Use this for hairpin barcoding. keep at 0 unless you know what you're doing" />
 
         <param name="KEEP_N" type="select" label="Discard sequences with unknown (N) bases">
             <option value="">Yes</option>
@@ -56,7 +58,7 @@
         </param>
     </inputs>
     <outputs>
-        <data format_source="input" name="output" metadata_source="input" />
+        <data name="output" format_source="input" metadata_source="input" />
     </outputs>
     <tests>
         <test>
@@ -71,7 +73,7 @@
             <output name="output" ftype="fastqsolexa" file="fastx_clipper1a.out" />
         </test>
     </tests>
-    <help>
+    <help><![CDATA[
 **What it does**
 
 This tool clips adapters from the 3'-end of the sequences in a FASTA/FASTQ file.
@@ -97,5 +99,6 @@
 This tool is based on `FASTX-toolkit`__ by Assaf Gordon.
 
  .. __: http://hannonlab.cshl.edu/fastx_toolkit/
-    </help>
+    ]]></help>
+    <expand macro="citations" />
 </tool>