You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@rave.apache.org by Marlon Pierce <mp...@cs.indiana.edu> on 2011/11/01 14:50:28 UTC
[discuss] Apache Rave 0.5-incubating Release Candidate
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
I was able to download and run the svn tagged version, source release, and the .tar.gz and .zip demo artifacts correctly. However, there seems to be a bug if I try to add a new gadget with the same URL twice. That is, I tried adding http://www.google.com/ig/modules/builtin_weather.xml with two different names. The first time worked correctly (or at least is waiting on admin approval). The second time produced an error ("Rave has suffered a brief meltdown"). I won't put the stack trace here.
Some other (probably non-blocking) issues:
* We need an obvious way to get to the admin interface from the front page or from the canonical user (or any other admin) account.
* If I approve a widget (change "preview" to "publish"), the form action should take me back to the widgets list or at least let me know if the action was successful or not.
* If I rate a widget, I can go to +2 or -2 rather than +1 or -1.
Marlon
-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.16 (Darwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQEcBAEBAgAGBQJOr/kkAAoJEEfVXEODPFID0q0H+wavhJTQB+ALUbOWvqM/Fht5
VVTWPd1iIMLcg0KkYQrhsGosDGKX3kpMzU8pC9LqT1wpqaajLIE+dO2BWSEnC54C
UM4o4t5XmBncUw8VcMqttl8O50tqp3bXhOLCQIf3s9TeGVYCC+sXcI8/9DzadNoz
7q9FPdysVd18yuD3jKx/TxeLIgSsFqSEXbk+A2UJkVAPJTgr+fOXALdoBByN92Cw
DNdm9el9kNF9iNLxS2MCCudUqQA2DLSaqa776Aj3awC4XUkSuut0p/uznHTRbD61
/eno5igk20PyMrHH+S4ezdCo6esJGn+GwybLgo0mHPhSLFy9Xhu75oSDuJF2t/s=
=clBo
-----END PGP SIGNATURE-----
RE: [discuss] Apache Rave 0.5-incubating Release Candidate
Posted by "Cooper, Sean D." <se...@mitre.org>.
Release
+1
* Admin interface
+1, but not a blocker
* Approve
+1, but not a blocker
* Rating
I am unable to reproduce on IE 8, FireFox 7.0.1, or Chrome 15.0.874.102, please open a new ticket with browser version specifics for testing
-Sean
-----Original Message-----
From: Marlon Pierce [mailto:mpierce@cs.indiana.edu]
Sent: Tuesday, November 01, 2011 9:50 AM
To: rave-dev@incubator.apache.org
Subject: [discuss] Apache Rave 0.5-incubating Release Candidate
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
I was able to download and run the svn tagged version, source release, and the .tar.gz and .zip demo artifacts correctly. However, there seems to be a bug if I try to add a new gadget with the same URL twice. That is, I tried adding http://www.google.com/ig/modules/builtin_weather.xml with two different names. The first time worked correctly (or at least is waiting on admin approval). The second time produced an error ("Rave has suffered a brief meltdown"). I won't put the stack trace here.
Some other (probably non-blocking) issues:
* We need an obvious way to get to the admin interface from the front page or from the canonical user (or any other admin) account.
* If I approve a widget (change "preview" to "publish"), the form action should take me back to the widgets list or at least let me know if the action was successful or not.
* If I rate a widget, I can go to +2 or -2 rather than +1 or -1.
Marlon
-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.16 (Darwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQEcBAEBAgAGBQJOr/kkAAoJEEfVXEODPFID0q0H+wavhJTQB+ALUbOWvqM/Fht5
VVTWPd1iIMLcg0KkYQrhsGosDGKX3kpMzU8pC9LqT1wpqaajLIE+dO2BWSEnC54C
UM4o4t5XmBncUw8VcMqttl8O50tqp3bXhOLCQIf3s9TeGVYCC+sXcI8/9DzadNoz
7q9FPdysVd18yuD3jKx/TxeLIgSsFqSEXbk+A2UJkVAPJTgr+fOXALdoBByN92Cw
DNdm9el9kNF9iNLxS2MCCudUqQA2DLSaqa776Aj3awC4XUkSuut0p/uznHTRbD61
/eno5igk20PyMrHH+S4ezdCo6esJGn+GwybLgo0mHPhSLFy9Xhu75oSDuJF2t/s=
=clBo
-----END PGP SIGNATURE-----
Re: [discuss] Apache Rave 0.5-incubating Release Candidate
Posted by Jasha Joachimsthal <j....@onehippo.com>.
On 1 November 2011 16:09, Carlucci, Tony <ac...@mitre.org> wrote:
> -----Original Message-----
> From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
> Sent: Tuesday, November 01, 2011 11:00 AM
> To: rave-dev@incubator.apache.org
> Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
>
> On 1 November 2011 15:36, Carlucci, Tony <ac...@mitre.org> wrote:
>
> >
> > -----Original Message-----
> > From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
> > Sent: Tuesday, November 01, 2011 10:10 AM
> > To: rave-dev@incubator.apache.org
> > Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
> >
> > On 1 November 2011 15:05, Ciancetta, Jesse E. <jc...@mitre.org> wrote:
> >
> > > >-----Original Message-----
> > > >From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
> > > >Sent: Tuesday, November 01, 2011 10:00 AM
> > > >To: rave-dev@incubator.apache.org
> > > >Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
> > > >
> > > >On 1 November 2011 14:50, Marlon Pierce <mp...@cs.indiana.edu>
> wrote:
> > > >
> > > >> -----BEGIN PGP SIGNED MESSAGE-----
> > > >> Hash: SHA1
> > > >>
> > > >> I was able to download and run the svn tagged version, source
> release,
> > > and
> > > >> the .tar.gz and .zip demo artifacts correctly. However, there seems
> > to
> > > be
> > > >> a bug if I try to add a new gadget with the same URL twice. That
> > is, I
> > > >> tried adding
> http://www.google.com/ig/modules/builtin_weather.xmlwith
> > > >> two different names. The first time worked correctly (or at least is
> > > >> waiting on admin approval). The second time produced an error ("Rave
> > has
> > > >> suffered a brief meltdown"). I won't put the stack trace here.
> > > >>
> > > >
> > > >The widget URL is unique to prevent duplicates. Until recently the
> user
> > > saw
> > > >a message in the add widget form that the widget already exists in the
> > > >database. We need to find out why the error page is shown now.
> > > >
> > >
> > > I don't personally view this as a blocker though since it only happens
> in
> > > a very specific case -- does anyone disagree?
> > >
> >
> > > It's not nice and should be fixed, but I don't see it as a blocker
> > either.
> >
> > > Jasha
> >
> > +1 on it needing to be fixed but not being a blocker.
> >
> > The error is occurring because: 1) DefaultWidgetService.registerNewWidget
> > returns a null Widget if the url already exists, and 2) the
> > RavePermissionEvaluator.hasPermission functions need to handle null
> objects
> > better. I'll create a bug ticket for this issue.
> >
>
> >I wrote the 1) logic but now I see it back I see room for improvement. The
> >Validator can reject the new widget if its url is already present. Then if
> >for some reason DefaultWidgetService#registerNewWidget is called for a URL
> >that already exists, it can throw a (DuplicateItem)Exception instead of
> >returning null. WDYT?
>
> >Jasha
>
> +1 on the DuplicateItemException enhancement - it's a good pattern to use
> across the board and easy for calling functions to trap and handle with
> "nice" client side messages. The hasPermission code needs to be fixed as
> well for better NPE prevention no matter what (RAVE-331).
>
> Tony
>
>
I created RAVE-332 for the change in logic.
Jasha
RE: [discuss] Apache Rave 0.5-incubating Release Candidate
Posted by "Carlucci, Tony" <ac...@mitre.org>.
-----Original Message-----
From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
Sent: Tuesday, November 01, 2011 11:00 AM
To: rave-dev@incubator.apache.org
Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
On 1 November 2011 15:36, Carlucci, Tony <ac...@mitre.org> wrote:
>
> -----Original Message-----
> From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
> Sent: Tuesday, November 01, 2011 10:10 AM
> To: rave-dev@incubator.apache.org
> Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
>
> On 1 November 2011 15:05, Ciancetta, Jesse E. <jc...@mitre.org> wrote:
>
> > >-----Original Message-----
> > >From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
> > >Sent: Tuesday, November 01, 2011 10:00 AM
> > >To: rave-dev@incubator.apache.org
> > >Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
> > >
> > >On 1 November 2011 14:50, Marlon Pierce <mp...@cs.indiana.edu> wrote:
> > >
> > >> -----BEGIN PGP SIGNED MESSAGE-----
> > >> Hash: SHA1
> > >>
> > >> I was able to download and run the svn tagged version, source release,
> > and
> > >> the .tar.gz and .zip demo artifacts correctly. However, there seems
> to
> > be
> > >> a bug if I try to add a new gadget with the same URL twice. That
> is, I
> > >> tried adding http://www.google.com/ig/modules/builtin_weather.xmlwith
> > >> two different names. The first time worked correctly (or at least is
> > >> waiting on admin approval). The second time produced an error ("Rave
> has
> > >> suffered a brief meltdown"). I won't put the stack trace here.
> > >>
> > >
> > >The widget URL is unique to prevent duplicates. Until recently the user
> > saw
> > >a message in the add widget form that the widget already exists in the
> > >database. We need to find out why the error page is shown now.
> > >
> >
> > I don't personally view this as a blocker though since it only happens in
> > a very specific case -- does anyone disagree?
> >
>
> > It's not nice and should be fixed, but I don't see it as a blocker
> either.
>
> > Jasha
>
> +1 on it needing to be fixed but not being a blocker.
>
> The error is occurring because: 1) DefaultWidgetService.registerNewWidget
> returns a null Widget if the url already exists, and 2) the
> RavePermissionEvaluator.hasPermission functions need to handle null objects
> better. I'll create a bug ticket for this issue.
>
>I wrote the 1) logic but now I see it back I see room for improvement. The
>Validator can reject the new widget if its url is already present. Then if
>for some reason DefaultWidgetService#registerNewWidget is called for a URL
>that already exists, it can throw a (DuplicateItem)Exception instead of
>returning null. WDYT?
>Jasha
+1 on the DuplicateItemException enhancement - it's a good pattern to use across the board and easy for calling functions to trap and handle with "nice" client side messages. The hasPermission code needs to be fixed as well for better NPE prevention no matter what (RAVE-331).
Tony
---
Anthony Carlucci | SW App Dev Eng, Sr. | R501 / KW App Development & Maint
e: acarlucci@mitre.org | v: 781.271.2432 | f: 781.271.3299
The MITRE Corporation | 202 Burlington Rd | Bedford, MA 01730-1420
Re: [discuss] Apache Rave 0.5-incubating Release Candidate
Posted by Ate Douma <at...@douma.nu>.
On 11/01/2011 03:59 PM, Jasha Joachimsthal wrote:
<snip/>
>>>>
>>>> The widget URL is unique to prevent duplicates. Until recently the user
>>> saw
>>>> a message in the add widget form that the widget already exists in the
>>>> database. We need to find out why the error page is shown now.
>>>>
>>>
>>> I don't personally view this as a blocker though since it only happens in
>>> a very specific case -- does anyone disagree?
>>>
>>
>>> It's not nice and should be fixed, but I don't see it as a blocker
>> either.
>>
>>> Jasha
>>
>> +1 on it needing to be fixed but not being a blocker.
>>
>> The error is occurring because: 1) DefaultWidgetService.registerNewWidget
>> returns a null Widget if the url already exists, and 2) the
>> RavePermissionEvaluator.hasPermission functions need to handle null objects
>> better. I'll create a bug ticket for this issue.
>>
>
> I wrote the 1) logic but now I see it back I see room for improvement. The
> Validator can reject the new widget if its url is already present. Then if
> for some reason DefaultWidgetService#registerNewWidget is called for a URL
> that already exists, it can throw a (DuplicateItem)Exception instead of
> returning null. WDYT?
>
> Jasha
>
+1 on the fix and +1 on regarding this as a non-blocker, so fine to fix for next
release.
I haven't yet reviewed the release itself, so this is not a vote for the
release, just concerns this issue.
Re: [discuss] Apache Rave 0.5-incubating Release Candidate
Posted by Marlon Pierce <mp...@cs.indiana.edu>.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
+1 also. Seems like the way to do it.
On 11/1/11 11:07 AM, Ciancetta, Jesse E. wrote:
>>> The error is occurring because: 1)
>>> DefaultWidgetService.registerNewWidget returns a null Widget if
>>> the url already exists, and 2) the
>>> RavePermissionEvaluator.hasPermission functions need to handle
>>> null
>> objects
>>> better. I'll create a bug ticket for this issue.
>>>
>>
>> I wrote the 1) logic but now I see it back I see room for
>> improvement. The Validator can reject the new widget if its url
>> is already present. Then if for some reason
>> DefaultWidgetService#registerNewWidget is called for a URL that
>> already exists, it can throw a (DuplicateItem)Exception instead
>> of returning null. WDYT?
>
> +1 -- sounds good to me.
>
>>
>> Jasha
-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.17 (Darwin)
Comment: GPGTools - http://gpgtools.org
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQEcBAEBAgAGBQJOsAtzAAoJEEfVXEODPFIDwZwIAIJ1Vk90hvX+5Y26U/JekSbF
r57YRjONQaeT0mYTl0RHw8Jdc79ThGO2Je4nlldQPRSKGavAoZLq8Q5GwHM/y3Ni
adnmCMls64i846ruP/YdjwW5fQ/SJYEn/2nckUvNcBxOJFxOzTfS+seAdvTkW8WK
ViiERvvTW2wjJBVYHXG/9IVRkV92tHCaZxfSEXN3oGQu7bWQiBoB7aVLutBG/5o+
JsPcVRtdh9bqnHAvq3RpF7smZntRk6t1Qz4hlxzDWLXjPltcr1YIUOzYRDgeubxi
gXmoiTV+TBy9Wo2M6QsnS6fNzMOqugpyNFuJZ8hoyB2k8QByFwS26EXOOzDLi54=
=RrFq
-----END PGP SIGNATURE-----
RE: [discuss] Apache Rave 0.5-incubating Release Candidate
Posted by "Ciancetta, Jesse E." <jc...@mitre.org>.
>> The error is occurring because: 1) DefaultWidgetService.registerNewWidget
>> returns a null Widget if the url already exists, and 2) the
>> RavePermissionEvaluator.hasPermission functions need to handle null
>objects
>> better. I'll create a bug ticket for this issue.
>>
>
>I wrote the 1) logic but now I see it back I see room for improvement. The
>Validator can reject the new widget if its url is already present. Then if
>for some reason DefaultWidgetService#registerNewWidget is called for a URL
>that already exists, it can throw a (DuplicateItem)Exception instead of
>returning null. WDYT?
+1 -- sounds good to me.
>
>Jasha
Re: [discuss] Apache Rave 0.5-incubating Release Candidate
Posted by Jasha Joachimsthal <j....@onehippo.com>.
On 1 November 2011 15:36, Carlucci, Tony <ac...@mitre.org> wrote:
>
> -----Original Message-----
> From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
> Sent: Tuesday, November 01, 2011 10:10 AM
> To: rave-dev@incubator.apache.org
> Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
>
> On 1 November 2011 15:05, Ciancetta, Jesse E. <jc...@mitre.org> wrote:
>
> > >-----Original Message-----
> > >From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
> > >Sent: Tuesday, November 01, 2011 10:00 AM
> > >To: rave-dev@incubator.apache.org
> > >Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
> > >
> > >On 1 November 2011 14:50, Marlon Pierce <mp...@cs.indiana.edu> wrote:
> > >
> > >> -----BEGIN PGP SIGNED MESSAGE-----
> > >> Hash: SHA1
> > >>
> > >> I was able to download and run the svn tagged version, source release,
> > and
> > >> the .tar.gz and .zip demo artifacts correctly. However, there seems
> to
> > be
> > >> a bug if I try to add a new gadget with the same URL twice. That
> is, I
> > >> tried adding http://www.google.com/ig/modules/builtin_weather.xmlwith
> > >> two different names. The first time worked correctly (or at least is
> > >> waiting on admin approval). The second time produced an error ("Rave
> has
> > >> suffered a brief meltdown"). I won't put the stack trace here.
> > >>
> > >
> > >The widget URL is unique to prevent duplicates. Until recently the user
> > saw
> > >a message in the add widget form that the widget already exists in the
> > >database. We need to find out why the error page is shown now.
> > >
> >
> > I don't personally view this as a blocker though since it only happens in
> > a very specific case -- does anyone disagree?
> >
>
> > It's not nice and should be fixed, but I don't see it as a blocker
> either.
>
> > Jasha
>
> +1 on it needing to be fixed but not being a blocker.
>
> The error is occurring because: 1) DefaultWidgetService.registerNewWidget
> returns a null Widget if the url already exists, and 2) the
> RavePermissionEvaluator.hasPermission functions need to handle null objects
> better. I'll create a bug ticket for this issue.
>
I wrote the 1) logic but now I see it back I see room for improvement. The
Validator can reject the new widget if its url is already present. Then if
for some reason DefaultWidgetService#registerNewWidget is called for a URL
that already exists, it can throw a (DuplicateItem)Exception instead of
returning null. WDYT?
Jasha
RE: [discuss] Apache Rave 0.5-incubating Release Candidate
Posted by "Carlucci, Tony" <ac...@mitre.org>.
-----Original Message-----
From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
Sent: Tuesday, November 01, 2011 10:10 AM
To: rave-dev@incubator.apache.org
Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
On 1 November 2011 15:05, Ciancetta, Jesse E. <jc...@mitre.org> wrote:
> >-----Original Message-----
> >From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
> >Sent: Tuesday, November 01, 2011 10:00 AM
> >To: rave-dev@incubator.apache.org
> >Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
> >
> >On 1 November 2011 14:50, Marlon Pierce <mp...@cs.indiana.edu> wrote:
> >
> >> -----BEGIN PGP SIGNED MESSAGE-----
> >> Hash: SHA1
> >>
> >> I was able to download and run the svn tagged version, source release,
> and
> >> the .tar.gz and .zip demo artifacts correctly. However, there seems to
> be
> >> a bug if I try to add a new gadget with the same URL twice. That is, I
> >> tried adding http://www.google.com/ig/modules/builtin_weather.xml with
> >> two different names. The first time worked correctly (or at least is
> >> waiting on admin approval). The second time produced an error ("Rave has
> >> suffered a brief meltdown"). I won't put the stack trace here.
> >>
> >
> >The widget URL is unique to prevent duplicates. Until recently the user
> saw
> >a message in the add widget form that the widget already exists in the
> >database. We need to find out why the error page is shown now.
> >
>
> I don't personally view this as a blocker though since it only happens in
> a very specific case -- does anyone disagree?
>
> It's not nice and should be fixed, but I don't see it as a blocker either.
> Jasha
+1 on it needing to be fixed but not being a blocker.
The error is occurring because: 1) DefaultWidgetService.registerNewWidget returns a null Widget if the url already exists, and 2) the RavePermissionEvaluator.hasPermission functions need to handle null objects better. I'll create a bug ticket for this issue.
> >>
> >>
> >> Some other (probably non-blocking) issues:
> >>
> >> * We need an obvious way to get to the admin interface from the front
> >page
> >> or from the canonical user (or any other admin) account.
> >>
> >
> >More than that: we need to work out how the menu should behave. Now we
> >build up the menu per JSP with items that may or may not be there.
> >
> >
> >>
> >> * If I approve a widget (change "preview" to "publish"), the form action
> >> should take me back to the widgets list or at least let me know if the
> >> action was successful or not.
> >>
> >
> >Sorry, I haven't added any feedback yet in the admin interface. If the
> >action was successful, you are now redirected back to the form.
> >
> >
> >>
> >> * If I rate a widget, I can go to +2 or -2 rather than +1 or -1.
> >>
> >
> >That's probably a display issue until you refresh the page. After the
> vote,
> >the number is updated client side until you refresh the page and you get
> >the real value from the server.
> >
> >
> >>
> >> Marlon
> >> -----BEGIN PGP SIGNATURE-----
> >> Version: GnuPG/MacGPG2 v2.0.16 (Darwin)
> >> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
> >>
> >>
> >iQEcBAEBAgAGBQJOr/kkAAoJEEfVXEODPFID0q0H+wavhJTQB+ALUbOWvqM/
> >Fht5
> >>
> >VVTWPd1iIMLcg0KkYQrhsGosDGKX3kpMzU8pC9LqT1wpqaajLIE+dO2BWSEnC
> >54C
> >>
> >UM4o4t5XmBncUw8VcMqttl8O50tqp3bXhOLCQIf3s9TeGVYCC+sXcI8/9DzadN
> >oz
> >>
> >7q9FPdysVd18yuD3jKx/TxeLIgSsFqSEXbk+A2UJkVAPJTgr+fOXALdoBByN92Cw
> >>
> >DNdm9el9kNF9iNLxS2MCCudUqQA2DLSaqa776Aj3awC4XUkSuut0p/uznHTRb
> >D61
> >>
> >/eno5igk20PyMrHH+S4ezdCo6esJGn+GwybLgo0mHPhSLFy9Xhu75oSDuJF2t/s
> >=
> >> =clBo
> >> -----END PGP SIGNATURE-----
> >>
>
---
Anthony Carlucci | SW App Dev Eng, Sr. | R501 / KW App Development & Maint
e: acarlucci@mitre.org | v: 781.271.2432 | f: 781.271.3299
The MITRE Corporation | 202 Burlington Rd | Bedford, MA 01730-1420
Re: [discuss] Apache Rave 0.5-incubating Release Candidate
Posted by Jasha Joachimsthal <j....@onehippo.com>.
On 1 November 2011 15:05, Ciancetta, Jesse E. <jc...@mitre.org> wrote:
> >-----Original Message-----
> >From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
> >Sent: Tuesday, November 01, 2011 10:00 AM
> >To: rave-dev@incubator.apache.org
> >Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
> >
> >On 1 November 2011 14:50, Marlon Pierce <mp...@cs.indiana.edu> wrote:
> >
> >> -----BEGIN PGP SIGNED MESSAGE-----
> >> Hash: SHA1
> >>
> >> I was able to download and run the svn tagged version, source release,
> and
> >> the .tar.gz and .zip demo artifacts correctly. However, there seems to
> be
> >> a bug if I try to add a new gadget with the same URL twice. That is, I
> >> tried adding http://www.google.com/ig/modules/builtin_weather.xml with
> >> two different names. The first time worked correctly (or at least is
> >> waiting on admin approval). The second time produced an error ("Rave has
> >> suffered a brief meltdown"). I won't put the stack trace here.
> >>
> >
> >The widget URL is unique to prevent duplicates. Until recently the user
> saw
> >a message in the add widget form that the widget already exists in the
> >database. We need to find out why the error page is shown now.
> >
>
> I don't personally view this as a blocker though since it only happens in
> a very specific case -- does anyone disagree?
>
It's not nice and should be fixed, but I don't see it as a blocker either.
Jasha
> >>
> >>
> >> Some other (probably non-blocking) issues:
> >>
> >> * We need an obvious way to get to the admin interface from the front
> >page
> >> or from the canonical user (or any other admin) account.
> >>
> >
> >More than that: we need to work out how the menu should behave. Now we
> >build up the menu per JSP with items that may or may not be there.
> >
> >
> >>
> >> * If I approve a widget (change "preview" to "publish"), the form action
> >> should take me back to the widgets list or at least let me know if the
> >> action was successful or not.
> >>
> >
> >Sorry, I haven't added any feedback yet in the admin interface. If the
> >action was successful, you are now redirected back to the form.
> >
> >
> >>
> >> * If I rate a widget, I can go to +2 or -2 rather than +1 or -1.
> >>
> >
> >That's probably a display issue until you refresh the page. After the
> vote,
> >the number is updated client side until you refresh the page and you get
> >the real value from the server.
> >
> >
> >>
> >> Marlon
> >> -----BEGIN PGP SIGNATURE-----
> >> Version: GnuPG/MacGPG2 v2.0.16 (Darwin)
> >> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
> >>
> >>
> >iQEcBAEBAgAGBQJOr/kkAAoJEEfVXEODPFID0q0H+wavhJTQB+ALUbOWvqM/
> >Fht5
> >>
> >VVTWPd1iIMLcg0KkYQrhsGosDGKX3kpMzU8pC9LqT1wpqaajLIE+dO2BWSEnC
> >54C
> >>
> >UM4o4t5XmBncUw8VcMqttl8O50tqp3bXhOLCQIf3s9TeGVYCC+sXcI8/9DzadN
> >oz
> >>
> >7q9FPdysVd18yuD3jKx/TxeLIgSsFqSEXbk+A2UJkVAPJTgr+fOXALdoBByN92Cw
> >>
> >DNdm9el9kNF9iNLxS2MCCudUqQA2DLSaqa776Aj3awC4XUkSuut0p/uznHTRb
> >D61
> >>
> >/eno5igk20PyMrHH+S4ezdCo6esJGn+GwybLgo0mHPhSLFy9Xhu75oSDuJF2t/s
> >=
> >> =clBo
> >> -----END PGP SIGNATURE-----
> >>
>
RE: [discuss] Apache Rave 0.5-incubating Release Candidate
Posted by "Ciancetta, Jesse E." <jc...@mitre.org>.
>-----Original Message-----
>From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
>Sent: Tuesday, November 01, 2011 10:00 AM
>To: rave-dev@incubator.apache.org
>Subject: Re: [discuss] Apache Rave 0.5-incubating Release Candidate
>
>On 1 November 2011 14:50, Marlon Pierce <mp...@cs.indiana.edu> wrote:
>
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> I was able to download and run the svn tagged version, source release, and
>> the .tar.gz and .zip demo artifacts correctly. However, there seems to be
>> a bug if I try to add a new gadget with the same URL twice. That is, I
>> tried adding http://www.google.com/ig/modules/builtin_weather.xml with
>> two different names. The first time worked correctly (or at least is
>> waiting on admin approval). The second time produced an error ("Rave has
>> suffered a brief meltdown"). I won't put the stack trace here.
>>
>
>The widget URL is unique to prevent duplicates. Until recently the user saw
>a message in the add widget form that the widget already exists in the
>database. We need to find out why the error page is shown now.
>
I don't personally view this as a blocker though since it only happens in a very specific case -- does anyone disagree?
>>
>>
>> Some other (probably non-blocking) issues:
>>
>> * We need an obvious way to get to the admin interface from the front
>page
>> or from the canonical user (or any other admin) account.
>>
>
>More than that: we need to work out how the menu should behave. Now we
>build up the menu per JSP with items that may or may not be there.
>
>
>>
>> * If I approve a widget (change "preview" to "publish"), the form action
>> should take me back to the widgets list or at least let me know if the
>> action was successful or not.
>>
>
>Sorry, I haven't added any feedback yet in the admin interface. If the
>action was successful, you are now redirected back to the form.
>
>
>>
>> * If I rate a widget, I can go to +2 or -2 rather than +1 or -1.
>>
>
>That's probably a display issue until you refresh the page. After the vote,
>the number is updated client side until you refresh the page and you get
>the real value from the server.
>
>
>>
>> Marlon
>> -----BEGIN PGP SIGNATURE-----
>> Version: GnuPG/MacGPG2 v2.0.16 (Darwin)
>> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
>>
>>
>iQEcBAEBAgAGBQJOr/kkAAoJEEfVXEODPFID0q0H+wavhJTQB+ALUbOWvqM/
>Fht5
>>
>VVTWPd1iIMLcg0KkYQrhsGosDGKX3kpMzU8pC9LqT1wpqaajLIE+dO2BWSEnC
>54C
>>
>UM4o4t5XmBncUw8VcMqttl8O50tqp3bXhOLCQIf3s9TeGVYCC+sXcI8/9DzadN
>oz
>>
>7q9FPdysVd18yuD3jKx/TxeLIgSsFqSEXbk+A2UJkVAPJTgr+fOXALdoBByN92Cw
>>
>DNdm9el9kNF9iNLxS2MCCudUqQA2DLSaqa776Aj3awC4XUkSuut0p/uznHTRb
>D61
>>
>/eno5igk20PyMrHH+S4ezdCo6esJGn+GwybLgo0mHPhSLFy9Xhu75oSDuJF2t/s
>=
>> =clBo
>> -----END PGP SIGNATURE-----
>>
Re: [discuss] Apache Rave 0.5-incubating Release Candidate
Posted by Jasha Joachimsthal <j....@onehippo.com>.
On 1 November 2011 14:50, Marlon Pierce <mp...@cs.indiana.edu> wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> I was able to download and run the svn tagged version, source release, and
> the .tar.gz and .zip demo artifacts correctly. However, there seems to be
> a bug if I try to add a new gadget with the same URL twice. That is, I
> tried adding http://www.google.com/ig/modules/builtin_weather.xml with
> two different names. The first time worked correctly (or at least is
> waiting on admin approval). The second time produced an error ("Rave has
> suffered a brief meltdown"). I won't put the stack trace here.
>
The widget URL is unique to prevent duplicates. Until recently the user saw
a message in the add widget form that the widget already exists in the
database. We need to find out why the error page is shown now.
>
>
> Some other (probably non-blocking) issues:
>
> * We need an obvious way to get to the admin interface from the front page
> or from the canonical user (or any other admin) account.
>
More than that: we need to work out how the menu should behave. Now we
build up the menu per JSP with items that may or may not be there.
>
> * If I approve a widget (change "preview" to "publish"), the form action
> should take me back to the widgets list or at least let me know if the
> action was successful or not.
>
Sorry, I haven't added any feedback yet in the admin interface. If the
action was successful, you are now redirected back to the form.
>
> * If I rate a widget, I can go to +2 or -2 rather than +1 or -1.
>
That's probably a display issue until you refresh the page. After the vote,
the number is updated client side until you refresh the page and you get
the real value from the server.
>
> Marlon
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG/MacGPG2 v2.0.16 (Darwin)
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
>
> iQEcBAEBAgAGBQJOr/kkAAoJEEfVXEODPFID0q0H+wavhJTQB+ALUbOWvqM/Fht5
> VVTWPd1iIMLcg0KkYQrhsGosDGKX3kpMzU8pC9LqT1wpqaajLIE+dO2BWSEnC54C
> UM4o4t5XmBncUw8VcMqttl8O50tqp3bXhOLCQIf3s9TeGVYCC+sXcI8/9DzadNoz
> 7q9FPdysVd18yuD3jKx/TxeLIgSsFqSEXbk+A2UJkVAPJTgr+fOXALdoBByN92Cw
> DNdm9el9kNF9iNLxS2MCCudUqQA2DLSaqa776Aj3awC4XUkSuut0p/uznHTRbD61
> /eno5igk20PyMrHH+S4ezdCo6esJGn+GwybLgo0mHPhSLFy9Xhu75oSDuJF2t/s=
> =clBo
> -----END PGP SIGNATURE-----
>