logo       

Re: urls in Plone with IE and Moz*: msg#00060

web.kupu.devel

Subject: Re: urls in Plone with IE and Moz*

yuppie wrote:

>> See http://codespeak.net/svn/kupu/trunk/kupu/plone/TODO.txt for a
>> list of things which might happen. Feedback on this list is most
>> welcome.
>
> What about merging plone's saveDataToField with the default
> prepareForm method?

They do look kind of similar, don't they?

The differences between saveDataToField and prepareForm:

prepareForm calls _serializeOutputToString, but saveDataToField does a lot
more messing about including the test to ignore nearly empty pages, and the
relative link conversion. The significant difference here is that
_serializeOutputToString returns a complete html document, but Plone only
wants the contents of the body.

We could make _serializeOutputToString check a config parameter to select
between returning the whole page or just the body part.

prepareForm takes an id (actually it calls it 'id' but it is a 'name') and
creates a textarea with that name which it appends onto the form.
saveDataToField takes a field and just sets its value. It should be pretty
easy to check that argument and if it is a string create the textarea, but
if it is a field just use it. It should probably also check if given a name
that there isn't already a field with that name and not create another if
there is one. [Query: where does the initial text for kupu come from if the
textarea doesn't already exist? Plone uses one textarea for the initial
value and the result]

Added to the growing Plone TODO list.


<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

News | FAQ | advertise