Mailing-List: contact dev-help@struts.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Struts Developers List" <dev@struts.apache.org>
Received-SPF: pass (nike.apache.org: domain of grobmeier@gmail.com designates
 209.85.216.169 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAMopvkNYo0Meag7vUDRehXtPYJ88oRhtZsw3ZWW9vhtD4f2HBQ@mail.gmail.com>
References: 
 <CAMopvkNU3azjeBh02-bTRiBUjevOCo-R7jkR29mWQXEKg=uHPA@mail.gmail.com>
 <4E187509.3080201@Newfield.org>
 <CAPFNckgRMDTpdc2-M6B_84U9u7qGLz8E+m-Uk6uc4dy7q8AynQ@mail.gmail.com>
 <4E1911EF.5000603@Newfield.org>
 <CAPFNckgFRRCO8YA1XhTZMK=6bRM02QRFVTvc=_TjyRpTvo1Asw@mail.gmail.com>
 <4E19E3FC.5050006@Newfield.org>
 <CAPFNcki9geXf_mrAEhH59wMs3sYk3M9uUtV0hKRwS8b7vOXw+w@mail.gmail.com>
 <4E1B92C6.10702@Newfield.org>
 <CAMopvkNYo0Meag7vUDRehXtPYJ88oRhtZsw3ZWW9vhtD4f2HBQ@mail.gmail.com>
From: Christian Grobmeier <grobmeier@gmail.com>
Date: Tue, 12 Jul 2011 13:44:20 +0200
Message-ID: 
 <CAPFNckhaH5KUeSFSRPPiY4B1PJ5X=DSqy0O-9u9dEQ5LeCFkSQ@mail.gmail.com>
Subject: Re: Comments in JSON
To: Struts Developers List <dev@struts.apache.org>, lukasz.lenart@gmail.com
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

> Thanks for all the comments and links, now I've got better overview
> what's the real issue here. My idea was the same, to add some flag to
> enable / disable comments in the output with default to true.

I could not find the place but as this is not standard json the
content type of the response header should not be "application/json".
It should be something else like "application/struts-json" or similar.
As I found out with a custom content type tools like jQuery might be
able to add a Converter which is able to correct the invalid string:
http://api.jquery.com/extending-ajax/#Converters

At the moment the default is application/json which is wrong, when the
enable comments option is true by default:
http://svn.apache.org/repos/asf/struts/struts2/trunk/plugins/json/src/main/=
java/org/apache/struts2/json/SerializationParams.java

I have added a similar feature to my own struts json plugin and
therefore I will implement such a converter. I will post the result
(if i manage :-)) later

Cheers,
Christian

>
> I'm going to implement it till weekend, if not, Dale you're welcome to
> do this as I'm going for short vacation.
>
>
> Regards
> --
> =C5=81ukasz
> + 48 606 323 122=C2=A0http://www.lenart.org.pl/
> Warszawa JUG conference - Confitura http://confitura.pl/
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> For additional commands, e-mail: dev-help@struts.apache.org
>
>


--=20
http://www.grobmeier.de

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org