plone.app.workflow 3.1.3 is broken

6 Messages

Hans-Peter Locher

Reply Threaded

Hi,
plone.app.workflow, as used in the latest Plone 3.1.3 (@@sharing) is
partially broken.

Using kss, it looses the currently selected settings from the form,
breaking local role settings on submit.

Best regards

mr_savage

[hans-peter_locher.vcf]

begin:vcard fn:Hans-Peter Locher n:Locher;Hans-Peter org:InQuant GmbH adr;quoted-printable:;;Bahnhofstra=C3=9Fe 11;Ravensburg;;88214;Deutschland email;internet:hans-peter.locher@... tel;work:+49 (0) 751 35 44 112 tel;fax:+49 (0) 751 35 44 115 x-mozilla-html:FALSE url:http://www.inquant.de version:2.1 end:vcard

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Plone-developers mailing list
Plone-developers@...
https://lists.sourceforge.net/lists/listinfo/plone-developers

Ramon Bartl

Re: plone.app.workflow 3.1.3 is broken

Reply Threaded

Hi mr_savage, hi plone developers,

seems like the variable 'roles' plone.app.workflow.browser.sharing L:
150 gets overwritten in plone.app.workflow.browser.sharing L:153...

Patch is in the Attachment...

greetz ramonski

Am 18.07.2008 um 11:53 schrieb Hans-Peter Locher:

> Hi,
> plone.app.workflow, as used in the latest Plone 3.1.3 (@@sharing) is
> partially broken.
>
> Using kss, it looses the currently selected settings from the form,
> breaking local role settings on submit.
>
> Best regards
>
> mr_savage
> <hans-
> peter_locher
> .vcf
> >
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's
> challenge
> Build the coolest Linux based applications with Moblin SDK & win
> great prizes
> Grand prize is a trip for two to an Open Source event anywhere in
> the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/_______________________________________________
> Plone-developers mailing list
> Plone-developers@...
> https://lists.sourceforge.net/lists/listinfo/plone-developers

sharing.diff (771 bytes) Download Attachment

Martin Aspeli

Re: plone.app.workflow 3.1.3 is broken

Reply Threaded

Ramon Bartl wrote:

Hi mr_savage, hi plone developers,

seems like the variable 'roles' plone.app.workflow.browser.sharing L:
150 gets overwritten in plone.app.workflow.browser.sharing L:153...

Patch is in the Attachment...

This sounds like the issue Laurence reported a few days ago: http://plone.org/support/forums/core#nabble-td535507

Wichert - I assume this is one for you to review and fix?

Martin

Wichert Akkerman

Re: plone.app.workflow 3.1.3 is broken

Reply Threaded

Previously Martin Aspeli wrote:

>
>
>
> Ramon Bartl wrote:
> >
> > Hi mr_savage, hi plone developers,
> >
> > seems like the variable 'roles' plone.app.workflow.browser.sharing L:
> > 150 gets overwritten in plone.app.workflow.browser.sharing L:153...
> >
> > Patch is in the Attachment...
> >
>
> This sounds like the issue Laurence reported a few days ago:
> http://plone.org/support/forums/core#nabble-td535507
>
> Wichert - I assume this is one for you to review and fix?

It was fixed many hours before your email.

Wichert.

--
Wichert Akkerman <wichert@...> It is simple to make things.
http://www.wiggy.net/ It is hard to make things simple.

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Plone-developers mailing list
Plone-developers@...
https://lists.sourceforge.net/lists/listinfo/plone-developers

Jens W. klein-2

Re: plone.app.workflow 3.1.3 is broken

Reply Threaded

Am Sat, 19 Jul 2008 17:05:11 +0200 schrieb Wichert Akkerman:
[..]
>> Ramon Bartl wrote:
[..]
>> > seems like the variable 'roles' plone.app.workflow.browser.sharing L:
>> > 150 gets overwritten in plone.app.workflow.browser.sharing L:153...
[..]
> It was fixed many hours before your email.

Can we please have a new release of plone.app.workflow? This is a stopper
for most intranets. http://pypi.python.org/pypi/plone.app.workflow still
lists the buggy 1.1.3 from July 7th as recent.

TIA ;) Jens

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Plone-developers mailing list
Plone-developers@...
https://lists.sourceforge.net/lists/listinfo/plone-developers

Ramon Bartl

Re: plone.app.workflow 3.1.3 is broken

Reply Threaded

Am 23.07.2008 um 11:49 schrieb Jens W. Klein:

> Am Sat, 19 Jul 2008 17:05:11 +0200 schrieb Wichert Akkerman:
> [..]
>>> Ramon Bartl wrote:
> [..]
>>>> seems like the variable 'roles'
>>>> plone.app.workflow.browser.sharing L:
>>>> 150 gets overwritten in plone.app.workflow.browser.sharing L:153...
> [..]
>> It was fixed many hours before your email.
>
> Can we please have a new release of plone.app.workflow? This is a
> stopper
> for most intranets. http://pypi.python.org/pypi/plone.app.workflow
> still
> lists the buggy 1.1.3 from July 7th as recent.
>
> TIA ;) Jens

+1

but there seems to be still a bug.

Users with inherited roles have this tiny little green icon displayed.
But when searching for another user, the icons for those users get
lost and unchecked checkboxes get displayed...

Didn't have time yet to analyse it, but seems to be only that the
'acquired' decorator gets lost after search

-ramonski

>
>
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's
> challenge
> Build the coolest Linux based applications with Moblin SDK & win
> great prizes
> Grand prize is a trip for two to an Open Source event anywhere in
> the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Plone-developers mailing list
> Plone-developers@...
> https://lists.sourceforge.net/lists/listinfo/plone-developers