[Plone-cvs] r35503 - plone.app.imaging/trunk/src/plone/app/imaging/profiles/default

[Andreas Z. <svn...@pl...>]

Author: witsch
Date: Tue Mar 23 11:20:20 2010
New Revision: 35503

Modified:
   plone.app.imaging/trunk/src/plone/app/imaging/profiles/default/propertiestool.xml
Log:
revert r34881: while re-installing the package and/or re-applying the profile doesn't create duplicate entries (the properties handler probably checks for this), changing one of the original values — let's say "mini 200:200" to "mini 300:300" — will add an extra entry to the properties.  it'll end up having two for "mini", after which the user would see correct results in some cases, and wrong ones in other, depending on which entry goes first.

it's rather unlikely that the default sizes will ever change, but this also applies if one would like to change on of the existing sizes in a site-specific profile, for example.  without setting a defined state and making sure `p.a.imaging`'s profile is applied first (which is easily done via `metadata.xml`) it's not clear which size definition will "win"...

Modified: plone.app.imaging/trunk/src/plone/app/imaging/profiles/default/propertiestool.xml
==============================================================================
--- plone.app.imaging/trunk/src/plone/app/imaging/profiles/default/propertiestool.xml	(original)
+++ plone.app.imaging/trunk/src/plone/app/imaging/profiles/default/propertiestool.xml	Tue Mar 23 11:20:20 2010
@@ -2,7 +2,7 @@
 <object name="portal_properties" meta_type="Plone Properties Tool">
  <object name="imaging_properties" meta_type="Plone Property Sheet">
   <property name="title">Image handling properties</property>
-  <property name="allowed_sizes" type="lines" purge="False">
+  <property name="allowed_sizes" type="lines">
    <element value="large 768:768"/>
    <element value="preview 400:400"/>
    <element value="mini 200:200"/>