[PATCH 1/5] Save info only from first cylinder on CSV export

Miika Turkia miika.turkia at gmail.com
Fri Dec 26 10:24:08 PST 2014


On Fri, Dec 26, 2014 at 6:42 PM, Dirk Hohndel <dirk at hohndel.org> wrote:

> Why only the first cylinder?
>

That is what we currently support in importing CSV logs. What would be a
sensible maximum amount of cylinders? This needs to be a fixed value so all
the exported dives have the same amount of fields...or we could have a
single field for cylinders that needs to be separately parsed and would
probably cause quite a bit of confusion.


> PS: and you may want to change your alias for the mailing list... you're
> still sending to the old domain... I don't expect that one to go away,
> like, ever, but I figured I should mention it :-)
>

I don't have anything that fancy, it is my bash history :D

miika

>
> On Fri, Dec 26, 2014 at 04:11:34PM +0200, Miika Turkia wrote:
> > Signed-off-by: Miika Turkia <miika.turkia at gmail.com>
> > ---
> >  xslt/xml2manualcsv.xslt | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/xslt/xml2manualcsv.xslt b/xslt/xml2manualcsv.xslt
> > index b549e95..bf7fb30 100644
> > --- a/xslt/xml2manualcsv.xslt
> > +++ b/xslt/xml2manualcsv.xslt
> > @@ -38,8 +38,8 @@
> >        </xsl:otherwise>
> >      </xsl:choose>
> >      <xsl:choose>
> > -      <xsl:when test="cylinder/@start|cylinder/@end != ''">
> > -        <xsl:apply-templates select="cylinder"/>
> > +      <xsl:when test="cylinder[1]/@start|cylinder[1]/@end != ''">
> > +        <xsl:apply-templates select="cylinder[1]"/>
> >        </xsl:when>
> >        <xsl:otherwise>
> >          <xsl:value-of select="$fs"/>
> > --
> > 1.9.1
> >
> > _______________________________________________
> > subsurface mailing list
> > subsurface at subsurface-divelog.org
> > http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20141226/56b85da5/attachment.html>


More information about the subsurface mailing list