Re: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
As the originator of management_page_charset, I should have jumped into this thread earlier. My appologies for my late arrival. On Thursday 15 January 2004 17:22, Martijn Faassen wrote: it assumes the only place management_page_charset can be coming from is a property, which is not true. Ive missed the use case that needs this. management_page_charset was added as a transition aid for people who already had sites where their use of the management interface assumed something other than latin-1 was used for plain string properties. That is, a transition towards using unicode objects for storing all natural language strings. I wouldnt like to see management_page_charset grow into something bigger than this - its too ugly. Considering the amount of rewriting necessary to make this saner and the scariness of doing that just before a Zope 2.7 release, I'll skip it and live with my Formulator hack solution. Can you point me to a desicription of this hack please. -- Toby Dickenson ___ Zope-Dev maillist - [EMAIL PROTECTED] http://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://mail.zope.org/mailman/listinfo/zope-announce http://mail.zope.org/mailman/listinfo/zope )
Re: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
Hi Hi guys - I was trying to be responsive to getting the issue resolved, since I'd like to make a (hopefully final) beta of 2.7 of Friday. I'll be happy to check in (or have you check in) whatever fixes are needed to give you the flexibility you need so long as it is b/w compatible, but I won't have the time to actively figure out what those fixes are this week. If you or Hajime can send me a patch against the current 2.7 branch, I'll make sure they get in before the beta is cut (or if either of you are committers it is also fine to checkin yourselves to the Zope-2_7-branch and head and let me know when its done). I have checkout 2.7 branch and test. And now seems good works for me. http://cvs.zope.org/Zope/lib/python/OFS/dtml/properties.dtml.diff?r1=1.8.18.6r2=1.8.18.7 Thanks a lot Brian. Is it good for you, Martijn ? Regards, Hajime Nakagami ___ Zope-Dev maillist - [EMAIL PROTECTED] http://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://mail.zope.org/mailman/listinfo/zope-announce http://mail.zope.org/mailman/listinfo/zope )
Re: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
Brian Lloyd wrote: I forward-ported these to the 2.7 branch the head. Any testing you can do to make sure I didn't break anything would be appreciated. I'm having trouble understanding what you forward-ported and what you'd like me to test. As far as I can determine management_page_header.dtml is still unchanged in CVS. Regards, Martijn ___ Zope-Dev maillist - [EMAIL PROTECTED] http://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://mail.zope.org/mailman/listinfo/zope-announce http://mail.zope.org/mailman/listinfo/zope )
Re: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
Brian Lloyd wrote: I forward-ported these to the 2.7 branch the head. Any testing you can do to make sure I didn't break anything would be appreciated. Now I understand that you were responding to these messages: I think the problem is same as reported by Kazuya Fukamachi http://mail.zope.org/pipermail/zope-dev/2003-December/021315.html and me. http://mail.zope.org/pipermail/zope-dev/2003-December/021338.html Fixing properties.dtml does not fix the problem that management_page_charset cannot be a callable anymore. This means that any logic to determine the right charset to use in displaying a page will fail to work (with an error) in Zope 2.7.(This can be fixed by rewriting the code to use a ComputedAttribute for management_page_charset.) So my problem and Hajime's may be different? Hajime? Regards, Martijn ___ Zope-Dev maillist - [EMAIL PROTECTED] http://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://mail.zope.org/mailman/listinfo/zope-announce http://mail.zope.org/mailman/listinfo/zope )
Re: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
Hi
Brian Lloyd wrote:
I forward-ported these to the 2.7 branch the head. Any testing
you can do to make sure I didn't break anything would be appreciated.
Now I understand that you were responding to these messages:
I think the problem is same as reported by Kazuya Fukamachi
http://mail.zope.org/pipermail/zope-dev/2003-December/021315.html
and me.
http://mail.zope.org/pipermail/zope-dev/2003-December/021338.html
Fixing properties.dtml does not fix the problem that
management_page_charset cannot be a callable anymore. This means that
any logic to determine the right charset to use in displaying a page
will fail to work (with an error) in Zope 2.7.(This can be fixed by
rewriting the code to use a ComputedAttribute for management_page_charset.)
So my problem and Hajime's may be different? Hajime?
Sorry I have not execute Zope 2.7 or HEAD now.
But I think needs not only the patch,
http://cvs.zope.org/Zope/lib/python/OFS/dtml/properties.dtml.diff?r1=1.14r2=1.15
but also below
--
Index: properties.dtml
===
RCS file: /cvs-repository/Zope/lib/python/OFS/dtml/properties.dtml,v
retrieving revision 1.15
diff -c -r1.15 properties.dtml
*** properties.dtml 14 Jan 2004 20:07:01 - 1.15
--- properties.dtml 15 Jan 2004 12:17:50 -
***
*** 5,10
--- 5,11
around. This means that this feature is not likely to be supported
in all future versions of zope.
/dtml-comment
+ dtml-call REQUEST.set( 'management_page_charset',management_page_charset)
dtml-call REQUEST.set('management_page_charset_tag','')
dtml-else
dtml-comment
___
Zope-Dev maillist - [EMAIL PROTECTED]
http://mail.zope.org/mailman/listinfo/zope-dev
** No cross posts or HTML encoding! **
(Related lists -
http://mail.zope.org/mailman/listinfo/zope-announce
http://mail.zope.org/mailman/listinfo/zope )
Re: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
Hajime Nakagami wrote: Hi Sorry I have not execute Zope 2.7 or HEAD now. But I think needs not only the patch, http://cvs.zope.org/Zope/lib/python/OFS/dtml/properties.dtml.diff?r1=1.14r2=1.15 but also below [patch to properties.dtml] To repeat: patching properties.dtml will never be able to solve the backwards compatibility breakage involving management_page_charset not being treated as a callable (if possible) anymore. It sounds therefore that Hajime and I are talking about two different things. Regards, Martijn ___ Zope-Dev maillist - [EMAIL PROTECTED] http://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://mail.zope.org/mailman/listinfo/zope-announce http://mail.zope.org/mailman/listinfo/zope )
RE: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
Hi guys - I was trying to be responsive to getting the issue resolved, since I'd like to make a (hopefully final) beta of 2.7 of Friday. I'll be happy to check in (or have you check in) whatever fixes are needed to give you the flexibility you need so long as it is b/w compatible, but I won't have the time to actively figure out what those fixes are this week. If you or Hajime can send me a patch against the current 2.7 branch, I'll make sure they get in before the beta is cut (or if either of you are committers it is also fine to checkin yourselves to the Zope-2_7-branch and head and let me know when its done). Brian Lloyd[EMAIL PROTECTED] V.P. Engineering 540.361.1716 Zope Corporation http://www.zope.com -Original Message- From: Martijn Faassen [mailto:[EMAIL PROTECTED] Behalf Of Martijn Faassen Sent: Thursday, January 15, 2004 6:41 AM To: Brian Lloyd Cc: Hajime Nakagami; [EMAIL PROTECTED] Subject: Re: [Zope-dev] 2.7 management_page_charset cannot be callable anymore Brian Lloyd wrote: I forward-ported these to the 2.7 branch the head. Any testing you can do to make sure I didn't break anything would be appreciated. Now I understand that you were responding to these messages: I think the problem is same as reported by Kazuya Fukamachi http://mail.zope.org/pipermail/zope-dev/2003-December/021315.html and me. http://mail.zope.org/pipermail/zope-dev/2003-December/021338.html Fixing properties.dtml does not fix the problem that management_page_charset cannot be a callable anymore. This means that any logic to determine the right charset to use in displaying a page will fail to work (with an error) in Zope 2.7.(This can be fixed by rewriting the code to use a ComputedAttribute for management_page_charset.) So my problem and Hajime's may be different? Hajime? Regards, Martijn ___ Zope-Dev maillist - [EMAIL PROTECTED] http://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://mail.zope.org/mailman/listinfo/zope-announce http://mail.zope.org/mailman/listinfo/zope )
Re: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
Brian Lloyd wrote:
I was trying to be responsive to getting the issue resolved, since
I'd like to make a (hopefully final) beta of 2.7 of Friday. I'll be
happy to check in (or have you check in) whatever fixes are needed
to give you the flexibility you need so long as it is b/w compatible,
but I won't have the time to actively figure out what those fixes
are this week.
Understood. I just spent some time gazing at what needs to be done
in properties.dtml (as well as manage_page_header.dtml), but frankly
the grotty DTML scares me and I already figured out a seemingly working
solution using ComputedAttribute in Formulator...
Fixing manage_page_header.dtml should be easy enough, just a one line fix:
dtml-call
RESPONSE.setHeader('content-type','text/html;charset='+_['management_page_charset'])
i.e. the use of _['management_page_charset']
properties.dtml however has some logic *before* manage_page_header gets
called, and doesn't seem to deal propertly with the case where
management_page_charset is manually set to UTF-8 (it doesn't go to
UTF-8 mode in that case, I think..), and it assumes the only place
management_page_charset can be coming from is a property, which is
not true.
Considering the amount of rewriting necessary to make this saner and the
scariness of doing that just before a Zope 2.7 release, I'll skip it
and live with my Formulator hack solution.
Regards,
Martijn
___
Zope-Dev maillist - [EMAIL PROTECTED]
http://mail.zope.org/mailman/listinfo/zope-dev
** No cross posts or HTML encoding! **
(Related lists -
http://mail.zope.org/mailman/listinfo/zope-announce
http://mail.zope.org/mailman/listinfo/zope )
RE: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
Brian Lloyd writes:
If you or Hajime can send me a patch against the current 2.7
branch, I'll make sure they get in before the beta is cut (or if
either of you are committers it is also fine to checkin yourselves
to the Zope-2_7-branch and head and let me know when its done).
Here is a patch that solves the issue with manage_page_charset not
beeing called if it is a method instead of a string-valued attribute:
Index: lib/python/App/dtml/manage_page_header.dtml
===
RCS file: /cvs-repository/Zope/lib/python/App/dtml/manage_page_header.dtml,v
retrieving revision 1.12
diff -w -u -r1.12 manage_page_header.dtml
--- lib/python/App/dtml/manage_page_header.dtml 22 Dec 2002 17:53:57 - 1.12
+++ lib/python/App/dtml/manage_page_header.dtml 15 Jan 2004 17:17:50 -
@@ -5,7 +5,7 @@
dtml-call REQUEST.set('management_page_charset','iso-8859-1')
/dtml-unless
meta http-equiv=content-type
content=text/html;charset=dtml-management_page_charset;
-dtml-call
RESPONSE.setHeader('content-type','text/html;charset='+management_page_charset)
+dtml-call
RESPONSE.setHeader('content-type','text/html;charset='+_.render(management_page_charset))
titledtml-if titledtml-title;/dtml-if/title
dtml-let ag=REQUEST.get('HTTP_USER_AGENT', '')
is_nav4=ag[:9] == 'Mozilla/4' and _.string.find(ag, 'MSIE') 0
I have not found any bad side effects after the patch, but I am not
very experienced with the soemwhat arcane DTML hacks, so it would be nice
if someone else does some testing with it.
At least it fulfills the requirements that manage_page_charset is
called if it is callable, and thus recovers Formulator.
What I am not certain about is if this reopens some XSS-security holes
the original change intended to close. However as long as one does not
have an utterly broken manage_page_charset method I cannot see why
this should happen.
Oh, I see Martijn already responded to the posting, so its maybe
superfluous anyway.
Cheers,
Clemens
___
Zope-Dev maillist - [EMAIL PROTECTED]
http://mail.zope.org/mailman/listinfo/zope-dev
** No cross posts or HTML encoding! **
(Related lists -
http://mail.zope.org/mailman/listinfo/zope-announce
http://mail.zope.org/mailman/listinfo/zope )
RE: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
-Original Message-
Sorry I have not execute Zope 2.7 or HEAD now.
But I think needs not only the patch,
http://cvs.zope.org/Zope/lib/python/OFS/dtml/properties.dtml.d
iff?r1=1.14r2=1.15
but also below
[patch to properties.dtml]
To repeat: patching properties.dtml will never be able to solve the
backwards compatibility breakage involving management_page_charset not
being treated as a callable (if possible) anymore.
It sounds therefore that Hajime and I are talking about two different
things.
dtml-if management_page_charset
dtml-comment
A site-global encoding specification in a property.
Note that this feature only works if there are no unicode
objects
around. This means that this feature is not likely to be
supported
in all future versions of zope.
/dtml-comment
+ dtml-call REQUEST.set(
'management_page_charset',management_page_charset)
dtml-call REQUEST.set('management_page_charset_tag','')
dtml-else
It seems we need one more line ;-)
Without this line, you cannot set management_page_charset form
ZMI-properties tab.
Thanks for all,
Kazuya Fukamachi
___
Zope-Dev maillist - [EMAIL PROTECTED]
http://mail.zope.org/mailman/listinfo/zope-dev
** No cross posts or HTML encoding! **
(Related lists -
http://mail.zope.org/mailman/listinfo/zope-announce
http://mail.zope.org/mailman/listinfo/zope )
[Zope-dev] 2.7 management_page_charset cannot be callable anymore
Hi there,
Some changes in Zope 2.7 break the possibility to make management_page_charset
a callable (for instance a method).
This breaks Formulator, as it uses this facility. This works just fine
in Zope 2.6, but breaks in Zope 2.7.
The silly thing is that Formulator 2.6.0 breaks in Zope 2.7 exactly because
it actually is the release that tries to do unicode *right* (while still
retaining backwards compatibility with older installations and offering
a non-unicode mode), and then Zope 2.7 makes it impossible.
I heard a report that a similar problem may be occuring with ZWiki..
The problem is in lib/python/App/dtml/manage_page_header.dtml:
dtml-unless management_page_charset
dtml-call REQUEST.set('management_page_charset','iso-8859-1')
/dtml-unless
meta http-equiv=content-type
content=text/html;charset=dtml-management_page_charset;
dtml-call
RESPONSE.setHeader('content-type','text/html;charset='+management_page_charset)
If I remember my DTML well, dtml-management_page_charset; should still call
the method if it's a callable, so that would be all right.
The next line however breaks, as it's going to treat my method as an attribute.
I think backwards compatibility got broken unintentionally here.. Could
this be restored? Using a ComputedAttribute for this would be rather
involved and it's possible other products are broken as well as a
result.
Regards,
Martijn
___
Zope-Dev maillist - [EMAIL PROTECTED]
http://mail.zope.org/mailman/listinfo/zope-dev
** No cross posts or HTML encoding! **
(Related lists -
http://mail.zope.org/mailman/listinfo/zope-announce
http://mail.zope.org/mailman/listinfo/zope )
Re: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
+1
I think the problem is same as reported by Kazuya Fukamachi
http://mail.zope.org/pipermail/zope-dev/2003-December/021315.html
and me.
http://mail.zope.org/pipermail/zope-dev/2003-December/021338.html
At least this is important problem in Japanese (maybe Korea and Chinese)
developpers.
Hi there,
Some changes in Zope 2.7 break the possibility to make management_page_charset
a callable (for instance a method).
This breaks Formulator, as it uses this facility. This works just fine
in Zope 2.6, but breaks in Zope 2.7.
The silly thing is that Formulator 2.6.0 breaks in Zope 2.7 exactly because
it actually is the release that tries to do unicode *right* (while still
retaining backwards compatibility with older installations and offering
a non-unicode mode), and then Zope 2.7 makes it impossible.
I heard a report that a similar problem may be occuring with ZWiki..
The problem is in lib/python/App/dtml/manage_page_header.dtml:
dtml-unless management_page_charset
dtml-call REQUEST.set('management_page_charset','iso-8859-1')
/dtml-unless
meta http-equiv=content-type
content=text/html;charset=dtml-management_page_charset;
dtml-call
RESPONSE.setHeader('content-type','text/html;charset='+management_page_charset)
If I remember my DTML well, dtml-management_page_charset; should still call
the method if it's a callable, so that would be all right.
The next line however breaks, as it's going to treat my method as an attribute.
I think backwards compatibility got broken unintentionally here.. Could
this be restored? Using a ComputedAttribute for this would be rather
involved and it's possible other products are broken as well as a
result.
Regards,
Martijn
___
Zope-Dev maillist - [EMAIL PROTECTED]
http://mail.zope.org/mailman/listinfo/zope-dev
** No cross posts or HTML encoding! **
(Related lists -
http://mail.zope.org/mailman/listinfo/zope-announce
http://mail.zope.org/mailman/listinfo/zope )
RE: [Zope-dev] 2.7 management_page_charset cannot be callable anymore
I forward-ported these to the 2.7 branch the head. Any testing
you can do to make sure I didn't break anything would be appreciated.
Brian Lloyd[EMAIL PROTECTED]
V.P. Engineering 540.361.1716
Zope Corporation http://www.zope.com
-Original Message-
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
Behalf Of Hajime Nakagami
Sent: Wednesday, January 14, 2004 1:29 PM
To: [EMAIL PROTECTED]
Subject: Re: [Zope-dev] 2.7 management_page_charset cannot be callable
anymore
+1
I think the problem is same as reported by Kazuya Fukamachi
http://mail.zope.org/pipermail/zope-dev/2003-December/021315.html
and me.
http://mail.zope.org/pipermail/zope-dev/2003-December/021338.html
At least this is important problem in Japanese (maybe Korea and Chinese)
developpers.
Hi there,
Some changes in Zope 2.7 break the possibility to make
management_page_charset
a callable (for instance a method).
This breaks Formulator, as it uses this facility. This works just fine
in Zope 2.6, but breaks in Zope 2.7.
The silly thing is that Formulator 2.6.0 breaks in Zope 2.7
exactly because
it actually is the release that tries to do unicode *right* (while still
retaining backwards compatibility with older installations and offering
a non-unicode mode), and then Zope 2.7 makes it impossible.
I heard a report that a similar problem may be occuring with ZWiki..
The problem is in lib/python/App/dtml/manage_page_header.dtml:
dtml-unless management_page_charset
dtml-call REQUEST.set('management_page_charset','iso-8859-1')
/dtml-unless
meta http-equiv=content-type
content=text/html;charset=dtml-management_page_charset;
dtml-call
RESPONSE.setHeader('content-type','text/html;charset='+management
_page_charset)
If I remember my DTML well, dtml-management_page_charset;
should still call
the method if it's a callable, so that would be all right.
The next line however breaks, as it's going to treat my method
as an attribute.
I think backwards compatibility got broken unintentionally here.. Could
this be restored? Using a ComputedAttribute for this would be rather
involved and it's possible other products are broken as well as a
result.
Regards,
Martijn
___
Zope-Dev maillist - [EMAIL PROTECTED]
http://mail.zope.org/mailman/listinfo/zope-dev
** No cross posts or HTML encoding! **
(Related lists -
http://mail.zope.org/mailman/listinfo/zope-announce
http://mail.zope.org/mailman/listinfo/zope )
___
Zope-Dev maillist - [EMAIL PROTECTED]
http://mail.zope.org/mailman/listinfo/zope-dev
** No cross posts or HTML encoding! **
(Related lists -
http://mail.zope.org/mailman/listinfo/zope-announce
http://mail.zope.org/mailman/listinfo/zope )
