- xwiki-devs@xwiki.org - lists.xwiki.org

[xwiki-devs] Fwd: [xwiki-notifications] r31169 - in platform/web/trunk/standard/src/main/webapp: resources/js/xwiki/editors resources/js/xwiki/usersandgroups templates

by Jerome Velociter

Hello, See below. ---------- Forwarded message ---------- From: abusenius <platform-notifications(a)xwiki.org> Date: Sun, Sep 19, 2010 at 2:37 PM Subject: [xwiki-notifications] r31169 - in platform/web/trunk/standard/src/main/webapp: resources/js/xwiki/editors resources/js/xwiki/usersandgroups templates To: platform-notifications(a)xwiki.org > Author: abusenius > Date: 2010-09-19 14:37:25 +0200 (Sun, 19 Sep 2010) > New Revision: 31169 > Modified: > platform/web/trunk/standard/src/main/webapp/resources/js/xwiki/editors/dataeditors.js > platform/web/trunk/standard/src/main/webapp/resources/js/xwiki/usersandgroups/usersandgroups.js > platform/web/trunk/standard/src/main/webapp/templates/addnewgroup.vm > platform/web/trunk/standard/src/main/webapp/templates/attachmentsinline.vm > platform/web/trunk/standard/src/main/webapp/templates/attachwysiwyg.vm > platform/web/trunk/standard/src/main/webapp/templates/changeavatar.vm > platform/web/trunk/standard/src/main/webapp/templates/changemyavatar.vm > platform/web/trunk/standard/src/main/webapp/templates/commentsinline.vm > platform/web/trunk/standard/src/main/webapp/templates/copy.vm > platform/web/trunk/standard/src/main/webapp/templates/delete.vm > platform/web/trunk/standard/src/main/webapp/templates/documentTags.vm > platform/web/trunk/standard/src/main/webapp/templates/edit.vm > platform/web/trunk/standard/src/main/webapp/templates/editclass.vm > platform/web/trunk/standard/src/main/webapp/templates/editgroup.vm > platform/web/trunk/standard/src/main/webapp/templates/editinline.vm > platform/web/trunk/standard/src/main/webapp/templates/editobject.vm > platform/web/trunk/standard/src/main/webapp/templates/edituser.vm > platform/web/trunk/standard/src/main/webapp/templates/editwysiwyg.vm > platform/web/trunk/standard/src/main/webapp/templates/editwysiwygnew.vm > platform/web/trunk/standard/src/main/webapp/templates/getgroupmembers.vm > platform/web/trunk/standard/src/main/webapp/templates/historyinline.vm > platform/web/trunk/standard/src/main/webapp/templates/importinline.vm > platform/web/trunk/standard/src/main/webapp/templates/passwd.vm > platform/web/trunk/standard/src/main/webapp/templates/recyclebinlist.vm > platform/web/trunk/standard/src/main/webapp/templates/registerinline.vm > platform/web/trunk/standard/src/main/webapp/templates/registeruser.vm > platform/web/trunk/standard/src/main/webapp/templates/renameStep1.vm > platform/web/trunk/standard/src/main/webapp/templates/resetversions.vm > platform/web/trunk/standard/src/main/webapp/templates/rightsUI.vm > platform/web/trunk/standard/src/main/webapp/templates/rollback.vm > platform/web/trunk/standard/src/main/webapp/templates/userinactive.vm > Log: > XWIKI-5462: Added CSRF tokens to all default templates [snip] > Modified: platform/web/trunk/standard/src/main/webapp/templates/editgroup.vm > =================================================================== > --- platform/web/trunk/standard/src/main/webapp/templates/editgroup.vm 2010-09-18 23:58:17 UTC (rev 31168) > +++ platform/web/trunk/standard/src/main/webapp/templates/editgroup.vm 2010-09-19 12:37:25 UTC (rev 31169) > @@ -71,7 +71,8 @@ > //<![CDATA[ > (function() { > var startup = function() { > - tabl = new ASSTable("$url", 15, "display1", "scrollbar1", "memberstable", displayMembers, true); > + var callback = function(row, i, table) { return displayMembers(row, i, table, form_token) }; > + tabl = new ASSTable("$url", 15, "display1", "scrollbar1", "memberstable", callback, true); I think something is missing here. For me the form_token you pass here to displayMembers is always undefined. Am I missing something ? Thanks, Jerome.

15 years

2
1
0 0

[xwiki-devs] Fwd: [xwiki-notifications] XWiki updates, 4 documents have been modified since 2010/09/17 00:00

by Jerome Velociter

Hi Anca, See below ---------- Forwarded message ---------- From: <notifications(a)xwiki.org> Date: Sat, Sep 18, 2010 at 12:00 AM Subject: [xwiki-notifications] XWiki updates, 4 documents have been modified since 2010/09/17 00:00 To: notifications(a)xwiki.org [snip] > lucaa - (16:59): I would need this to be able to implement the Main.Dashboard page as a dashboard with gadgets > lucaa - (17:01): any opinions? > jvdrean - (17:03): lucaa: what do you need exactly ? having macros in the default enterprise won’t resolve the PR problem right ? > lucaa - (17:03): well, i would like to implement the Main.Dashboard as a dashboard with gadgets > lucaa - (17:04): and since gadgets are macros, the components of the Main.Dashboard (spaces, recent changes, tag cloud) should be macros > lucaa - (17:04): now, they could be java macros, but the problem is that they will be harder to modify > lucaa - (17:04): which I don’t like, in general > lucaa - (17:05): another solution is to use the {{include }} macros and leave them as pages... > lucaa - (17:05): but > lucaa - (17:05): in this case > lucaa - (17:05): they won’t be configurable using macro parameters > lucaa - (17:06): the good part though, is that anyone with edit rights would be able to change them Actually only global visibility (across wikis) needs programming rights. Local wiki visibility only requires the user to have edit on the macro document. See org.xwiki.rendering.internal.macro.wikibridge.DefaultWikiMacroManager#isAllowed We still need to agree we want to have wiki macros in the default wiki. Jerome. [snip]

15 years

1
0
0 0

[xwiki-devs] [proposal] Add a document XWiki.ChangePassword and deprecate the passwd.vm template.

by Caleb James DeLisle

To fix "XWIKI-5469: Change password feature does not use validation" ( http://jira.xwiki.org/jira/browse/XWIKI-5469 ) I would like to add a document for helping the user change their password. passwd.vm is only referenced in one place in XWiki.XWikiUserPreferencesSheet so as a template it is unused unless the Enterprise XAR is installed. To avoid a "rat's nest" dependency graph, I do not like templates to have statically defined document names so I'd like to avoid the method used in registerinline.vm. Also any proposal which moves code from the /templates/ directory to the inside of wikiland has my +1. WDYT? Caleb

15 years

7
6
0 0

[xwiki-devs] [proposal] Add xcontext.dropPermissions() function to take away all programming/script author permissions when called.

by Caleb James DeLisle

I think this function would be useful, I have used code like it to make sure scripts didn't run with PR when they shouldn't and I think it deserves a place in the api. The function doesn't need any special privileges to run, but after it's run, api.Api#hasProgrammingRights will always return false and api.Document#saveAsAuthor will be attempting to save as the guest instead of the real document author. (likewise with getDocumentAsAuthor and deleteAsAuthor) Once permissions have been dropped, they cannot be regained (for the duration of the request.) This depends on another function in api.Api which is package private. String getEffectiveScriptAuthorName() returns the name given by XWikiContext.getDoc().getAuthor() unless dropPermissions() has allready been called or XWikiContext.getDoc() == null in which case it returns the guest username. Why in the old api.Context? I envision in the future a SecurityManager component will exist which distrusts not only script but core java as well. I think the option to drop privileges when unneeded should also be present in the final security manager implementation. Since security managers require a large secure code base to be useful, I am proposing api.Context#dropPermissions() as an interim solution. WDYT? Caleb

15 years

3
4
0 0

Re: [xwiki-devs] [xwiki-notifications] r31124 - platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources

by Vincent Massol

Hi Alex, On Sep 15, 2010, at 8:00 PM, abusenius (SVN) wrote: > Author: abusenius > Date: 2010-09-15 20:00:02 +0200 (Wed, 15 Sep 2010) > New Revision: 31124 > > Modified: > platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm > Log: > XWIKI-5461: Added configuration for CSRF protection > > Modified: platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm > =================================================================== > --- platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm 2010-09-15 17:59:36 UTC (rev 31123) > +++ platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm 2010-09-15 18:00:02 UTC (rev 31124) > @@ -267,3 +267,23 @@ > #crypto.passwd.keyDerivationFunctionPropertiesForPasswordVerification = millisecondsOfProcessorTimeToSpend = 200 > #crypto.passwd.keyDerivationFunctionPropertiesForPasswordVerification = numberOfKilobytesOfMemoryToUse = 1024 > #crypto.passwd.keyDerivationFunctionPropertiesForPasswordVerification = derivedKeyLength = 32 > + > +#------------------------------------------------------------------------------------- > +# CSRF token component > +#------------------------------------------------------------------------------------- > + > +#-# [Since 2.5M2] > +#-# Controls whether secret token validation mechanism should be used (to prevent CSRF attacks). > +#-# > +#-# If enabled, all actions requiring "comment", "edit", "delete", "admin" or "programming" rights > +#-# will check that the parameter "form_token" with the value of a random secret token is present > +#-# in the request. > +#-# This feature requires CSRFToken component. I think we could remove this last sentence since the CSRFToken component is bundled with the platform and this message will probably confuse the user who's not going to know how to check if he has this component or not. It'll make him/her ask himself questions, which we don't want IMO. > +#-# > +#-# Valid values: > +#-# 0: Disabled > +#-# 1: Enabled > +#-# > +#-# Default value is 0 > +# core.csrftoken.enabled = 0 I guess we're going to turn it on by default when the implementation is finished? Thanks -Vincent

15 years

2
1
0 0

[xwiki-devs] [discussion] Remove XWatch modules for building standalone distribution

by Anca Luca

Hi devs, I updated locally XWiki Watch trunk to work with XWiki Enterprise 2.3.2 (right now it's 1.8.4 version) and I would like to commit this in the Watch trunk. There is a little issue though, with the modules for building the standalone distribution(s) (distribution, database, installers, web), which need to be handled, for which there are 3 options: 0/ update the distribution related modules to make them compatible with 2.3 and the process is not straightforward due to Watch particularities (this would mean extra work and since I don't know when I will be able to do it, it means not commit right away) 1A/ remove the distribution related modules from the watch trunk and leave it installable as a .xar + component (as described at http://watch.xwiki.org/xwiki/bin/view/Main/Installation#HXWikiWatchoveranex… ) drawback is that users installing it need to edit the web.xml which is not that easy for everyone. 1B/ leave the distribution related modules, just exclude them from the default build since they will fail, until 0/ can be fixed right. I would go for 1B since I think partial update to 2.3 is better than nothing (current version on 1.8.4 is not really useful in most cases) and earlier is better than later. WDYT? Thanks, Anca

15 years

5
6
0 0

[xwiki-devs] [VOTE] Move component related events to their own project

by Thomas Mortagne

Hi devs, For http://jira.xwiki.org/jira/browse/XWIKI-5490 I need to make ObservationManager itself listen to ComponentDescriptorAddedEvent events. There is two issue here: - currently ComponentDescriptorAddedEvent is in the same project than the component manager implemenation which is pretty much the same thing as behing internal since there is no reason to force to depends on a specific implementation of component manager just to listen on some generic events that has nothing to do with it - i can move them to component-api because that would produce a circular dependency with observation project which depends on component-api So the only technical solution I can see is to create a new project with theses events. I propose to name it xwiki-core-component-observation and to move ComponentEventManager from component-api to it too. Note: i took ObservationManager here because it's a core one but a lot of other components will probably have the exact same need to support properly installed/uninstalled extensions WDYT ? -- Thomas Mortagne

15 years

4
4
0 0

[xwiki-devs] XEclipse part of URL to xwiki is hardcoded

by Andriy Vovchyna

Hi, We meet one unpleasant issue with current XEclipse (standalone version <http://forge.objectweb.org/project/download.php?group_id=170&file_id=11944> xwiki-eclipse-rcp-win32-win32-x86-1.2-rc-1.zip) We tried to use XEclipse for our xwiki based application browsing. But when we are trying to find some object stored in a page we are getting error "This data manager is connected to an XWiki that does not support object management." After deep investigation we found problem place in XEclipse code. There is hardcoded "expectation" of "xwiki" string in URL of connection in DataNamager.Connect() http://svn.xwiki.org/svnroot/xwiki/xeclipse/trunk/plugins/org.xwiki.eclipse. core/src/main/java/org/xwiki/eclipse/core/DataManager.java. public void connect() throws XWikiEclipseException, CoreException { ..... if (serverInfo.getBaseUrl().contains("xwiki")) { ...... } else { /* We are talking to a confluence server */ .... supportedFunctionalities.remove(Functionality.OBJECTS); ..... } .... } So, simple fixing of problem is renaming of web context on xwiki web server (e.g. war file renaming) with adding "xwiki" string. Are there plans to move this "hardcoding" on the connection configuration level (UI options like version selections - xwiki, confluence etc.)? Thanks, Andriy V.

15 years

1
0
0 0

by Marius Dumitru Florea

On 09/15/2010 06:34 PM, mflorea (SVN) wrote: > Author: mflorea > Date: 2010-09-15 17:34:58 +0200 (Wed, 15 Sep 2010) > New Revision: 31119 [snip] > Property changes on: platform/web/branches/xwiki-web-2.4/standard > ___________________________________________________________________ > Added: svn:mergeinfo > + /platform/web/trunk/standard:31118 > > > Property changes on: platform/web/branches/xwiki-web-2.4/standard/src/main/webapp/resources/icons/silk > ___________________________________________________________________ > Deleted: svn:mergeinfo > - > > > Property changes on: platform/web/branches/xwiki-web-2.4/standard/src/main/webapp/resources/js/prototype/prototype.js > ___________________________________________________________________ > Deleted: svn:mergeinfo > - > > > Property changes on: platform/web/branches/xwiki-web-2.4/standard/src/main/webapp/resources/js/scriptaculous > ___________________________________________________________________ > Deleted: svn:mergeinfo > - > > > Property changes on: platform/web/branches/xwiki-web-2.4/standard/src/main/webapp/resources/js/xwiki/accordion/accordion.js > ___________________________________________________________________ > Deleted: svn:mergeinfo > - > > > Property changes on: platform/web/branches/xwiki-web-2.4/standard/src/main/webapp/resources/js/xwiki/chartwizard/chwSampleBarChart.png > ___________________________________________________________________ > Deleted: svn:mergeinfo > - ..do you know how can we get rid of this? It makes the diff almost unreadable. Thanks, Marius

15 years

1
0
0 0

[xwiki-devs] [VOTE] Separate observation API and implementation

by Thomas Mortagne

Hi devs, Following "[VOTE] Move component related events to their own project" I also need to separate api and implementation in observation project because component-observation depends on observation api and i need to make observation manager listen to component events which mean depending on component-observation. Here is my +1 Thanks, -- Thomas Mortagne

15 years

6
5
0 0

Re: [xwiki-devs] [foaf-protocols] DNS problem with foafssl.org -> myxwiki

by Henry Story

I seem to have DNS problems on myxwiki. I can't yet tell if this is a problem with foafssl.org or with myxwiki.org, but I think it is with xwiki. Below is an extract of what Dirk (Chief Architect of the BBC) has to say On 14 Sep 2010, at 21:08, Dirk-Willem van Gulik wrote: > Op 14 sep 2010, om 21:59 heeft Henry Story het volgende geschreven: > >> That has not been updated for a while it seems, since 2000 at least >> http://sourceforge.net/projects/dnswalk/files/ > > I suspect that DNS has not changed that much since then. > >> # dig ns myxwiki.org >> >> ; <<>> DiG 9.3.3rc2 <<>> ns myxwiki.org > > Ok. > >> ;; flags: qr rd ra; QUERY: 1, ANSWER: 3, AUTHORITY: 0, ADDITIONAL: 3 > > So an answer; but whatever answers is not authoritive. >> >> myxwiki.org. 8 IN NS lanai.xpertnet.biz. >> myxwiki.org. 8 IN NS maui.xpertnet.biz. >> myxwiki.org. 8 IN NS ns-fr.xpertnet.biz. >> > These should be auhtoriative. >> >> # dig -t any @maui.xpertnet.biz > > So you are asking these. > >> ;; flags: qr rd; QUERY: 1, ANSWER: 0, AUTHORITY: 13, ADDITIONAL: 0 > > And no answer. > >> ;; AUTHORITY SECTION: >> . 518400 IN NS A.ROOT-SERVERS.NET. >> > And defers you to the root servers. And it is sure of that. > > -> conclusion - this guy is not configured as your dns server; i.e. it does not know your zone. > >> # dig -t any @ns-fr.xpertnet.biz >> [[ AT THIS POINT IT HANGS FOR A WHILE]] >> ; <<>> DiG 9.3.3rc2 <<>> -t any @ns-fr.xpertnet.biz >> ; (1 server found) >> ;; global options: printcmd >> ;; connection timed out; no servers could be reached > > And this guy is not running at all. Or in other words - your entire domain is foobar - but luckily not with huge TTL's or anything - so it is not hard to recover; either have the authoritive servers fixed - or move it somewhere else. If you are stuck - let me know - happy to run some zones over here or pull zones from SVN/hidden-master. If I run this from a server in Ireland I get $ dig -t any @ns-fr.xpertnet.biz ; <<>> DiG 9.6.0-APPLE-P2 <<>> -t any @ns-fr.xpertnet.biz ; (1 server found) ;; global options: +cmd ;; Got answer: ;; ->>HEADER<<- opcode: QUERY, status: REFUSED, id: 55036 ;; flags: qr rd; QUERY: 1, ANSWER: 0, AUTHORITY: 0, ADDITIONAL: 0 ;; WARNING: recursion requested but not available ;; QUESTION SECTION: ;. IN NS ;; Query time: 39 msec ;; SERVER: 188.165.123.33#53(188.165.123.33) ;; WHEN: Tue Sep 14 22:31:55 2010 ;; MSG SIZE rcvd: 17 Which sounds like the problem is the same here. > > Dw. Social Web Architect http://bblfish.net/

15 years

1
0
0 0

[xwiki-devs] I'd like to block clierr from breaking the build (Re: [xwiki-notifications] r31093 - in platform/core/trunk/xwiki-core: . src/main/java/com/xpn/xwiki/api)

by Caleb James DeLisle

I removed a method which was inaccessible to any code which might use it: protected BaseProperty getBaseProperty() and made final a field which is not altered anywhere. Assuming nobody has extended c.x.x.api.* these are not api breaks. I would like to block clierr from breaking the build rather than merging out this change. WDYT? Caleb cjdelisle (SVN) wrote: > Author: cjdelisle > Date: 2010-09-14 12:03:32 +0200 (Tue, 14 Sep 2010) > New Revision: 31093 > > Modified: > platform/core/trunk/xwiki-core/pom.xml > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/api/Element.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/api/Property.java > Log: > [cleanup] Documented c.x.x.api.Property, removed an unnecessary override and an unusable method, and made an un-reassignable field final. > > Modified: platform/core/trunk/xwiki-core/pom.xml > =================================================================== > --- platform/core/trunk/xwiki-core/pom.xml 2010-09-14 09:14:38 UTC (rev 31092) > +++ platform/core/trunk/xwiki-core/pom.xml 2010-09-14 10:03:32 UTC (rev 31093) > @@ -896,7 +896,6 @@ > **/api/Document.java, > **/api/DocumentSection.java, > **/api/Object.java, > - **/api/Property.java, > **/api/Util.java, > **/api/XWiki.java, > **/atom/lifeblog/LifeblogServices.java, > > Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/api/Element.java > =================================================================== > --- platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/api/Element.java 2010-09-14 09:14:38 UTC (rev 31092) > +++ platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/api/Element.java 2010-09-14 10:03:32 UTC (rev 31093) > @@ -33,7 +33,7 @@ > public class Element extends Api > { > /** The internal element which this wraps. */ > - protected BaseElement element; > + protected final BaseElement element; > > /** > * The Constructor. > > Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/api/Property.java > =================================================================== > --- platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/api/Property.java 2010-09-14 09:14:38 UTC (rev 31092) > +++ platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/api/Property.java 2010-09-14 10:03:32 UTC (rev 31093) > @@ -18,28 +18,33 @@ > * 02110-1301 USA, or see the FSF site: http://www.fsf.org. > * > */ > + > package com.xpn.xwiki.api; > > import com.xpn.xwiki.XWikiContext; > import com.xpn.xwiki.objects.BaseProperty; > > +/** > + * Property is a single attribute of an XWiki {@link com.xpn.xwiki.api.Object}. > + * > + * @version $Id$ > + */ > public class Property extends Element > { > + /** > + * The Constructor. > + * > + * @param property the internal {@link com.xpn.xwiki.objects.BaseProperty} to wrap. > + * @param context the XWikiContext which may be used to get information about the current request. > + */ > public Property(BaseProperty property, XWikiContext context) > { > super(property, context); > } > > - protected BaseProperty getBaseProperty() > - { > - return (BaseProperty) element; > - } > - > - public String getName() > - { > - return element.getName(); > - } > - > + /** > + * @return the internal {@link com.xpn.xwiki.objects.BaseProperty} which this Property wraps. > + */ > public BaseProperty getProperty() > { > if (hasProgrammingRights()) { > @@ -49,8 +54,12 @@ > } > } > > + /** > + * @return the actual value of the property, as a String, Number or List. > + */ > public java.lang.Object getValue() > { > + // This is evil, any property which happens to be called 'password' will be masked. TODO fix. > if (element.getName().equals("password") > && !getXWikiContext().getWiki().getRightService().hasProgrammingRights( > getXWikiContext())) { > > _______________________________________________ > notifications mailing list > notifications(a)xwiki.org > http://lists.xwiki.org/mailman/listinfo/notifications >

15 years

2
1
0 0

[xwiki-devs] [vote] Integrate CSRF protection into XWiki 2.5

by Alex Busenius

Hi devs, I've been working on a CSRF protection mechanism for quite some time. It is based on so called secret tokens (also called nonces) that are included into forms and links and checked on server side. The implementation allows to resubmit a failed request (e.g. in case the user is logged out in the meanwhile), so that the data is not lost in case of bugs. JIRA issue: http://jira.xwiki.org/jira/browse/XWIKI-4873 Component implementation: http://svn.xwiki.org/svnroot/xwiki/contrib/sandbox/xwiki-csrftoken/ Old proposal: http://lists.xwiki.org/pipermail/devs/2010-March/017727.html I think it is time to move the CSRF component to the main repository and start using it everywhere. The protection will be disabled by default until all bugs are fixed. The steps to do would be: 1. Move CSRF token component to platform 2. Fix all templates to use CSRF tokens 3. Fix all applications to use CSRF tokens 4. Fix all actions to check CSRF tokens 5. Fix all integration tests to work with enabled CSRF protection I have patches for steps 2-4, but NOT for 5. Many (about 30-40 last time I counted) integration tests still fail with enabled CSRF protection, because they (mis)use CSRF to set up tests (edit/create pages). Here is my +1 WDYT? Thanks, Alex

15 years

5
6
0 0

[xwiki-devs] Extension Manager first working POC !

by Thomas Mortagne

Hi xwikiers, Since May and actively since end of August I'm working on a POC for the future (long awaited) Extension Manager. It's now in a state where you can play with it a bit ! So here is a quick advertisement. You can find it on http://svn.xwiki.org/svnroot/xwiki/contrib/sandbox/xwiki-extension/ What it does already: - display remote extension and its dependencies as a tree - install remote extension and its dependencies from repositories - framework support any kind of extension using ExtensionInstaller component interface but only jar installer is implemented right now - framework support any kind of repository using RepositoryFactory component interface but only maven is implemented right now - load installed application when XWiki starts Critical features not yet supported: - miscellaneous rights protections in extension manager script service - uninstall/upgrade support: need to find a way to remove a jar from classloader. Note that you can remove an extension from local repository and restart. - xar installer: just need to provide a ExtensionInstaller implementation with role hint "xar" since this needs access to the whole model the first version will probably be implemented with old APIs (XWikiDocument, etc.) on xwiki-core side - install extension not coming from a repository: the current API is very repository centric but you can do it "by hand" by putting directly the extension jar file in the local repository. - real descriptors in local repository for installed extensions (to store in a repository independent way extension information as well as some local only metadas like the fact that an extension as been installed as a dependency of another etc.) - others on http://svn.xwiki.org/svnroot/xwiki/contrib/sandbox/xwiki-extension/README.t… I'm preparing some design documentation that will go on http://dev.xwiki.org/xwiki/bin/view/Design/ExtensionManagerProposal but i don't have much time right now so it will probably be tomorrow (GMT). The current state is not final at all but it still introduce a lot of APIs and design, lets call it an over designed POC. It needs to be discussed in all it's aspects before having its experimental tag removed. The plan is to distribute it with XE coming with 2.5 whatever it's state (when it does not represent a risk for XE obviously) as an experimental playground with all the required warnings in the UI and promote it when we are happy with a first version. Thanks, -- Thomas Mortagne

15 years

3
3
0 0

Re: [xwiki-devs] [xwiki-notifications] r31088 - platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources

by Vincent Massol

Although this makes sense from a code and organisation POV, I don't think it makes sense from a user POV. Personally I'd prefer to keep the old style since it's less confusing for users and more logical. I know there's a risk of namespace clash but it's very small and can be solved by changing property names if need be. What do others think? I'm not too hung up on this but I want to make it as easy as possible for users and prevent the "WTF?" feeling (aka as the "don't make me think" feeling) :) Thanks -Vincent On Sep 14, 2010, at 9:00 AM, mflorea (SVN) wrote: > Author: mflorea > Date: 2010-09-14 09:00:57 +0200 (Tue, 14 Sep 2010) > New Revision: 31088 > > Modified: > platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm > Log: > XTCONFRES-50: Add configuration for including image dimensions in the image URL and for limiting image dimensions > * Changed "rendering." prefix to "rendering.xwiki." for XWiki-specific rendering configuration properties > > > Modified: platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm > =================================================================== > --- platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm 2010-09-14 06:58:39 UTC (rev 31087) > +++ platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm 2010-09-14 07:00:57 UTC (rev 31088) > @@ -68,7 +68,7 @@ > #-# downloaded, improving thus the page loading speed. > #-# > #-# Default value is true. > -# rendering.includeImageDimensionsInImageURL = true > +# rendering.xwiki.imageDimensionsIncludedInImageURL = true > > #-# [Since 2.5M2] > #-# One way to improve page load speed is to resize images on the server side just before rendering the page. The > @@ -79,13 +79,13 @@ > #-# The default value is -1 which means image width is not limited by default. Use a value greater than 0 to limit the > #-# image width (pixels). Note that the aspect ratio is kept even when both the width and the height of the image are > #-# limited. > -# rendering.imageWidthLimit = 1024 > -# rendering.imageWidthLimit = -1 > +# rendering.xwiki.imageWidthLimit = 1024 > +# rendering.xwiki.imageWidthLimit = -1 > > #-# [Since 2.5M2] > #-# See rendering.imageWidthLimit > -# rendering.imageHeightLimit = 768 > -# rendering.imageHeightLimit = -1 > +# rendering.xwiki.imageHeightLimit = 768 > +# rendering.xwiki.imageHeightLimit = -1 > > #------------------------------------------------------------------------------------- > # Cache >

15 years

3
2
0 0

Re: [xwiki-devs] [xwiki-notifications] r31048 - platform/web/trunk/standard/src/main/webapp/templates

by Vincent Massol

Hi Alex, All commits that fix something or make some changes (except refactoring, ie same behavior, just internal way the code is written) must reference a jira issue. If this problem was caused by some changes in the trunk that wasn't released yet, you should use the same jira issue. If it's a fix over some previously released version then it needs a new jira issue. The idea is that a user who looks at the release notes must be able to see what was modified vs the version he has. Thanks -Vincent On Sep 11, 2010, at 7:15 PM, abusenius (SVN) wrote: > Author: abusenius > Date: 2010-09-11 19:15:10 +0200 (Sat, 11 Sep 2010) > New Revision: 31048 > > Modified: > platform/web/trunk/standard/src/main/webapp/templates/editobject.vm > Log: > [misc] Fixed broken URL > > Modified: platform/web/trunk/standard/src/main/webapp/templates/editobject.vm > =================================================================== > --- platform/web/trunk/standard/src/main/webapp/templates/editobject.vm 2010-09-11 02:32:40 UTC (rev 31047) > +++ platform/web/trunk/standard/src/main/webapp/templates/editobject.vm 2010-09-11 17:15:10 UTC (rev 31048) > @@ -16,7 +16,7 @@ > #macro(displayObject $obj $class $props $firstfield) > <div id="xobject_${escapetool.xml($class.name)}_${obj.number}" class="xobject"> > <div id="xobject_${escapetool.xml($class.name)}_${obj.number}_title" class="xobject-title"><h3>#cleanClassname(${class.name}) ${obj.number}: $!{escapetool.xml($doc.display($firstfield, $obj))} > - <a href="$doc.getURL('objectremove', "classname=${escapetool.url($class.name)}&classid=${obj.number}&xredirect=${doc.getURL('edit', 'editor=object')}")" class="xobject-action delete" title="$msg.get('core.editors.object.removeObject.tooltip')">$msg.get('core.editors.object.removeObject')</a> > + <a href="$doc.getURL('objectremove', "classname=${escapetool.url($class.name)}&classid=${obj.number}&xredirect=${escapetool.url($doc.getURL('edit', 'editor=object'))}")" class="xobject-action delete" title="$msg.get('core.editors.object.removeObject.tooltip')">$msg.get('core.editors.object.removeObject')</a> > #if(!$hasCustomObject) <a href="$doc.getURL("edit", "editor=object&classname=${escapetool.url($class.name)}&object=${obj.number}")" class="xobject-action edit" title="$msg.get('core.editors.object.editSingleObject.tooltip')">$msg.get('core.editors.object.editSingleObject')</a> #end > </h3></div> > <div id="xobject_${escapetool.xml($class.name)}_${obj.number}_content" class="xobject-content"><div>

15 years

2
1
0 0

Re: [xwiki-devs] [xwiki-notifications] r31053 - in platform/core/trunk/xwiki-rendering: xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki xwiki-rendering-xwiki xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration xwiki-rendering-xwiki/src/main/java/org/x

by Vincent Massol

On Sep 12, 2010, at 1:30 PM, mflorea (SVN) wrote: > Author: mflorea > Date: 2010-09-12 13:30:15 +0200 (Sun, 12 Sep 2010) > New Revision: 31053 > > Modified: > platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml > platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java > Log: > XWIKI-5479: Include image dimensions in the image URL for server side image resizing > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -50,7 +50,7 @@ > * </ul> > * Note that if the page title is empty or not defined then it defaults to %p. This is also the case > * if the title cannot be retrieved for the document. > - * > + * > * The default is "%p". Some examples: "%s.%p", "%w:%s.%p". > * > * @return the format to use to display link labels when the user hasn't specified a label > @@ -63,4 +63,42 @@ > * descriptor (ie defined by the macro author) will be used > */ > Properties getMacroCategories(); > + > + /** > + * @return {@code true} to include the image dimensions extracted from the image parameters in the image URL, > + * {@code false} otherwise; when image dimensions are included in the image URL the image can be resized > + * on the server side before being downloaded. > + * @since 2.5M2 > + */ > + boolean isIncludeImageDimensionsInImageURL(); I think is should be better named: isImageDimensionsIncludedInImageURL() I haven't followed too much but why do we need a flag for this? Why not always have the dimensions in the URL? Also, I agree with Thomas that this config should be internal to the xwiki implementation and not generic to all implementations. Thanks -Vincent > + > + /** > + * One way to improve page load speed is to resize images on the server side just before rendering the page. The > + * rendering module can use the image width provided by the user to scale the image. When the user doesn't specify > + * the image width the rendering module can limit the width of the image based on this configuration parameter. > + * <p> > + * The default value is {@code -1} which means image width is not limited by default. Use a value greater than 0 to > + * limit the image width (pixels). Note that the aspect ratio is kept even when both the width and the height of the > + * image are limited. > + * > + * @return the maximum image width when there's no user supplied width > + * @see #isIncludeImageDimensionsInImageURL() > + * @since 2.5M2 > + */ > + int getImageWidthLimit(); > + > + /** > + * One way to improve page load speed is to resize images on the server side just before rendering the page. The > + * rendering module can use the image height provided by the user to scale the image. When the user doesn't specify > + * the image height the rendering module can limit the height of the image based on this configuration parameter. > + * <p> > + * The default value is {@code -1} which means image height is not limited by default. Use a value greater than 0 to > + * limit the image height (pixels). Note that the aspect ratio is kept even when both the width and the height of > + * the image are limited. > + * > + * @return the maximum image height when there's no user supplied height > + * @see #isIncludeImageDimensionsInImageURL() > + * @since 2.5M2 > + */ > + int getImageHeightLimit(); > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -45,6 +45,21 @@ > private Properties macroCategories = new Properties(); > > /** > + * @see #getImageWidthLimit() > + */ > + private int imageWidthLimit = -1; > + > + /** > + * @see #getImageHeightLimit() > + */ > + private int imageHeightLimit = -1; > + > + /** > + * @see #isIncludeImageDimensionsInImageURL() > + */ > + private boolean includeImageDimensionsInImageURL = true; > + > + /** > * {@inheritDoc} > * > * @see org.xwiki.rendering.configuration.RenderingConfiguration#getLinkLabelFormat() > @@ -84,4 +99,66 @@ > // to work even without a configuration store. > this.macroCategories.setProperty(macroId.toString(), category); > } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageWidthLimit() > + */ > + public int getImageWidthLimit() > + { > + return this.imageWidthLimit; > + } > + > + /** > + * @param imageWidthLimit the maximum image width when there's no user supplied width > + */ > + public void setImageWidthLimit(int imageWidthLimit) > + { > + // This method is useful for those using the XWiki Rendering in standalone mode since it allows the rendering > + // to work even without a configuration store. > + this.imageWidthLimit = imageWidthLimit; > + } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageHeightLimit() > + */ > + public int getImageHeightLimit() > + { > + return this.imageHeightLimit; > + } > + > + /** > + * @param imageHeightLimit the maximum image height when there's no user supplied height > + */ > + public void setImageHeightLimit(int imageHeightLimit) > + { > + // This method is useful for those using the XWiki Rendering in standalone mode since it allows the rendering > + // to work even without a configuration store. > + this.imageHeightLimit = imageHeightLimit; > + } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#isIncludeImageDimensionsInImageURL() > + */ > + public boolean isIncludeImageDimensionsInImageURL() > + { > + return this.includeImageDimensionsInImageURL; > + } > + > + /** > + * @param includeImageDimensionsInImageURL {@code true} to include image dimensions extracted from the image > + * parameters in the image URL so that the image can be resized on the server side before being > + * downloaded, {@code false} otherwise > + */ > + public void setIncludeImageDimensionsInImageURL(boolean includeImageDimensionsInImageURL) > + { > + // This method is useful for those using the XWiki Rendering in standalone mode since it allows the rendering > + // to work even without a configuration store. > + this.includeImageDimensionsInImageURL = includeImageDimensionsInImageURL; > + } > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -42,10 +42,9 @@ > * Default implementation for rendering images as XHTML. We handle both cases: > * <ul> > * <li>when inside a wiki (ie when an implementation of {@link WikiModel} is provided.</li> > - * <li>when outside of a wiki. In this case we only handle external images and document images don't > - * display anything.</li> > + * <li>when outside of a wiki. In this case we only handle external images and document images don't display anything.</li> > * </ul> > - * > + * > * @version $Id$ > * @since 2.0M3 > */ > @@ -68,6 +67,7 @@ > > /** > * {@inheritDoc} > + * > * @see Initializable#initialize() > */ > public void initialize() throws InitializationException > @@ -83,7 +83,7 @@ > > /** > * {@inheritDoc} > - * > + * > * @see XHTMLImageRenderer#setXHTMLWikiPrinter(XHTMLWikiPrinter) > */ > public void setXHTMLWikiPrinter(XHTMLWikiPrinter printer) > @@ -103,7 +103,7 @@ > > /** > * {@inheritDoc} > - * > + * > * @see XHTMLImageRenderer#onImage(Image, boolean, Map) > */ > public void onImage(Image image, boolean isFreeStandingURI, Map<String, String> parameters) > @@ -118,8 +118,9 @@ > // that would not honor this contract. > if (this.wikiModel != null) { > DocumentImage documentImage = DocumentImage.class.cast(image); > - imageURL = this.wikiModel.getAttachmentURL(documentImage.getDocumentName(), > - documentImage.getAttachmentName()); > + imageURL = > + this.wikiModel.getImageURL(documentImage.getDocumentName(), documentImage.getAttachmentName(), > + parameters); > } else { > throw new RuntimeException("Invalid Image type. In non wiki mode, all image types must be URL images."); > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -19,13 +19,15 @@ > */ > package org.xwiki.rendering.wiki; > > +import java.util.Map; > + > import org.xwiki.component.annotation.ComponentRole; > > /** > * Bridge between the Rendering module and a Wiki Model. Contains wiki APIs required by Rendering classes such as > - * Renderers. For example the XHTML Link Renderer needs to know if a wiki document exists in order to know how to > - * generate the HTML (in order to display a question mark for non existing documents) and it also needs to get the > - * URL pointing the wiki document. > + * Renderers. For example the XHTML Link Renderer needs to know if a wiki document exists in order to know how to > + * generate the HTML (in order to display a question mark for non existing documents) and it also needs to get the URL > + * pointing the wiki document. > * > * @version $Id$ > * @since 2.0M1 > @@ -34,36 +36,48 @@ > public interface WikiModel > { > /** > - * @param documentName the document name of the document containing the attachment. The syntax used depends on > - * the underlying wiki system used. For example for XWiki a valid documentName would be > - * {@code wiki:Space.Page} > + * @param documentName the document name of the document containing the attachment. The syntax used depends on the > + * underlying wiki system used. For example for XWiki a valid documentName would be > + * {@code wiki:Space.Page} > * @param attachmentName the name of the attachment in the passed document wikip page > * @return the URL to the attachment > */ > String getAttachmentURL(String documentName, String attachmentName); > > /** > - * @param documentName the document name as a String. The syntax used depends on > - * the underlying wiki system used. For example for XWiki a valid documentName would be > - * {@code wiki:Space.Page} > + * Generate image specific URL. The difference with {@link #getAttachmentURL(String, String)} is that in some > + * implementation we want to make a distinction between displayed image and a simple link targeting an attachment > + * file. > + * > + * @param documentName the document name of the document containing the attachment. The syntax used depends on the > + * underlying wiki system used. For example for XWiki a valid documentName would be > + * {@code wiki:Space.Page} > + * @param attachmentName the name of the attachment in the passed document wiki page > + * @param parameters custom parameters > + * @return the URL to the image > + * @since 2.5M2 > + */ > + String getImageURL(String documentName, String attachmentName, Map<String, String> parameters); > + > + /** > + * @param documentName the document name as a String. The syntax used depends on the underlying wiki system used. > + * For example for XWiki a valid documentName would be {@code wiki:Space.Page} > * @return true if the document exists and can be viewed or false otherwise > */ > boolean isDocumentAvailable(String documentName); > > /** > - * @param documentName the document name as a String. The syntax used depends on > - * the underlying wiki system used. For example for XWiki a valid documentName would be > - * {@code wiki:Space.Page} > + * @param documentName the document name as a String. The syntax used depends on the underlying wiki system used. > + * For example for XWiki a valid documentName would be {@code wiki:Space.Page} > * @param anchor an anchor pointing to some place inside the document or null > * @param queryString a query string specifying some parameters or null > * @return the URL to view the specified wiki document > */ > String getDocumentViewURL(String documentName, String anchor, String queryString); > - > + > /** > - * @param documentName the document name as a String. The syntax used depends on > - * the underlying wiki system used. For example for XWiki a valid documentName would be > - * {@code wiki:Space.Page} > + * @param documentName the document name as a String. The syntax used depends on the underlying wiki system used. > + * For example for XWiki a valid documentName would be {@code wiki:Space.Page} > * @param anchor an anchor pointing to some place inside the document or null > * @param queryString a query string specifying some parameters or null > * @return the URL to edit the specified wiki document > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml 2010-09-12 11:30:15 UTC (rev 31053) > @@ -54,6 +54,12 @@ > <artifactId>xwiki-core-script</artifactId> > <version>${pom.version}</version> > </dependency> > + <dependency> > +  > + <groupId>net.sourceforge.cssparser</groupId> > + <artifactId>cssparser</artifactId> > + <version>0.9.5</version> > + </dependency> >  > <dependency> > <groupId>org.xwiki.platform</groupId> > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -19,13 +19,13 @@ > */ > package org.xwiki.rendering.internal.configuration; > > +import java.util.Properties; > + > import org.xwiki.component.annotation.Component; > import org.xwiki.component.annotation.Requirement; > import org.xwiki.configuration.ConfigurationSource; > import org.xwiki.rendering.configuration.RenderingConfiguration; > > -import java.util.Properties; > - > /** > * All configuration options for the rendering subsystem. > * > @@ -46,7 +46,7 @@ > private static final String DEFAULT_LINK_LABEL_FORMAT = "%p"; > > /** > - * Defines from where to read the rendering configuration data. > + * Defines from where to read the rendering configuration data. > */ > @Requirement > private ConfigurationSource configuration; > @@ -58,16 +58,46 @@ > */ > public String getLinkLabelFormat() > { > - return this.configuration.getProperty(PREFIX + "linkLabelFormat", DEFAULT_LINK_LABEL_FORMAT); > + return this.configuration.getProperty(PREFIX + "linkLabelFormat", DEFAULT_LINK_LABEL_FORMAT); > } > > /** > * {@inheritDoc} > - * > + * > * @see org.xwiki.rendering.configuration.RenderingConfiguration#getMacroCategories() > */ > public Properties getMacroCategories() > { > return this.configuration.getProperty(PREFIX + "macroCategories", Properties.class); > } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageWidthLimit() > + */ > + public int getImageWidthLimit() > + { > + return this.configuration.getProperty(PREFIX + "imageWidthLimit", -1); > + } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageHeightLimit() > + */ > + public int getImageHeightLimit() > + { > + return this.configuration.getProperty(PREFIX + "imageHeightLimit", -1); > + } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#isIncludeImageDimensionsInImageURL() > + */ > + public boolean isIncludeImageDimensionsInImageURL() > + { > + return this.configuration.getProperty(PREFIX + "includeImageDimensionsInImageURL", true); > + } > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -19,17 +19,25 @@ > */ > package org.xwiki.rendering.internal.wiki; > > +import java.io.IOException; > +import java.io.StringReader; > import java.io.UnsupportedEncodingException; > import java.net.URLEncoder; > +import java.util.Map; > > import org.apache.commons.lang.StringUtils; > +import org.w3c.css.sac.InputSource; > +import org.w3c.dom.css.CSSStyleDeclaration; > import org.xwiki.bridge.DocumentAccessBridge; > -import org.xwiki.model.reference.DocumentReference; > import org.xwiki.component.annotation.Component; > import org.xwiki.component.annotation.Requirement; > +import org.xwiki.model.reference.DocumentReference; > import org.xwiki.model.reference.EntityReferenceSerializer; > +import org.xwiki.rendering.configuration.RenderingConfiguration; > import org.xwiki.rendering.wiki.WikiModel; > > +import com.steadystate.css.parser.CSSOMParser; > + > /** > * Implementation using the Document Access Bridge ({@link DocumentAccessBridge}). > * > @@ -39,14 +47,47 @@ > @Component > public class XWikiWikiModel implements WikiModel > { > + /** > + * The suffix used to mark an amount of pixels. > + */ > + private static final String PIXELS = "px"; > + > + /** > + * The name of the {@code width} image parameter. > + */ > + private static final String WIDTH = "width"; > + > + /** > + * The name of the {@code height} image parameter. > + */ > + private static final String HEIGHT = "height"; > + > + /** > + * The component used to access configuration parameters. > + */ > @Requirement > + private RenderingConfiguration renderingConfiguration; > + > + /** > + * The component used to access the underlying XWiki model. > + */ > + @Requirement > private DocumentAccessBridge documentAccessBridge; > > + /** > + * The component used to serialize entity references to strings. > + */ > @Requirement > private EntityReferenceSerializer<String> entityReferenceSerializer; > > /** > + * The object used to parse the CSS from the image style parameter. > + */ > + private final CSSOMParser cssParser = new CSSOMParser(); > + > + /** > * {@inheritDoc} > + * > * @see WikiModel#getAttachmentURL(String, String) > */ > public String getAttachmentURL(String documentName, String attachmentName) > @@ -55,7 +96,112 @@ > } > > /** > + * Extracts the specified image dimension from the image parameters. > + * > + * @param dimension either {@code width} or {@code height} > + * @param imageParameters the image parameters; may include the {@code width}, {@code height} and {@code style} > + * parameters > + * @return the value of the passed dimension if it is specified in the image parameters, {@code null} otherwise > + */ > + private String getImageDimension(String dimension, Map<String, String> imageParameters) > + { > + // Check first if the style parameter contains information about the given dimension. In-line style has priority > + // over the dimension parameters. > + String value = null; > + String style = imageParameters.get("style"); > + if (StringUtils.isNotBlank(style)) { > + try { > + CSSStyleDeclaration sd = cssParser.parseStyleDeclaration(new InputSource(new StringReader(style))); > + value = sd.getPropertyValue(dimension); > + } catch (IOException e) { > + // Ignore the style parameter. > + } > + } > + if (StringUtils.isBlank(value)) { > + // Fall back on the value of the dimension parameter. > + value = imageParameters.get(dimension); > + } > + return value; > + } > + > + /** > + * Creates the query string that can be added to an image URL to resize the image on the server side. > + * > + * @param imageParameters image parameters, including width and height then they are specified > + * @return the query string to be added to an image URL in order to resize the image on the server side > + */ > + private StringBuilder getImageURLQueryString(Map<String, String> imageParameters) > + { > + String width = StringUtils.chomp(getImageDimension(WIDTH, imageParameters), PIXELS); > + String height = StringUtils.chomp(getImageDimension(HEIGHT, imageParameters), PIXELS); > + boolean useHeight = StringUtils.isNotEmpty(height) && StringUtils.isNumeric(height); > + StringBuilder queryString = new StringBuilder(); > + if (StringUtils.isEmpty(width) || !StringUtils.isNumeric(width)) { > + // Width is unspecified or is not measured in pixels. > + if (useHeight) { > + // Height is specified in pixels. > + queryString.append('&').append(HEIGHT).append('=').append(height); > + } else { > + // If image width and height are unspecified or if they are not expressed in pixels then limit the image > + // size to best fit the rectangle specified in the configuration (keeping aspect ratio). > + int widthLimit = renderingConfiguration.getImageWidthLimit(); > + if (widthLimit > 0) { > + queryString.append('&').append(WIDTH).append('=').append(widthLimit); > + } > + int heightLimit = renderingConfiguration.getImageHeightLimit(); > + if (heightLimit > 0) { > + queryString.append('&').append(HEIGHT).append('=').append(heightLimit); > + } > + if (widthLimit > 0 && heightLimit > 0) { > + queryString.append("&keepAspectRatio=").append(true); > + } > + } > + } else { > + // Width is specified in pixels. > + queryString.append('&').append(WIDTH).append('=').append(width); > + if (useHeight) { > + // Height is specified in pixels. > + queryString.append('&').append(HEIGHT).append('=').append(height); > + } > + } > + return queryString; > + } > + > + /** > * {@inheritDoc} > + * > + * @see org.xwiki.rendering.wiki.WikiModel#getImageURL(java.lang.String, java.lang.String, java.util.Map) > + */ > + public String getImageURL(String documentName, String attachmentName, Map<String, String> parameters) > + { > + String url = getAttachmentURL(documentName, attachmentName); > + if (!renderingConfiguration.isIncludeImageDimensionsInImageURL()) { > + return url; > + } > + > + StringBuilder queryString = getImageURLQueryString(parameters); > + if (queryString.length() == 0) { > + return url; > + } > + > + // Determine the insertion point. > + int insertionPoint = url.lastIndexOf('#'); > + if (insertionPoint < 0) { > + // No fragment identifier. > + insertionPoint = url.length(); > + } > + if (url.lastIndexOf('?', insertionPoint) < 0) { > + // No query string. > + queryString.setCharAt(0, '?'); > + } > + > + // Insert the query string. > + return new StringBuilder(url).insert(insertionPoint, queryString).toString(); > + } > + > + /** > + * {@inheritDoc} > + * > * @see WikiModel#isDocumentAvailable(String) > */ > public boolean isDocumentAvailable(String documentName) > @@ -65,6 +211,7 @@ > > /** > * {@inheritDoc} > + * > * @see WikiModel#getDocumentViewURL(String, String, String) > */ > public String getDocumentViewURL(String documentName, String anchor, String queryString) > @@ -74,6 +221,7 @@ > > /** > * {@inheritDoc} > + * > * @see WikiModel#getDocumentEditURL(String, String, String) > */ > public String getDocumentEditURL(String documentName, String anchor, String queryString) > @@ -89,17 +237,17 @@ > // See http://www.w3.org/TR/html40/appendix/notes.html#non-ascii-chars > // TODO: Once the xwiki-url module is usable, refactor this code to use it and remove the need to > // perform explicit encoding here. > - modifiedQueryString = > + modifiedQueryString = > "parent=" + URLEncoder.encode(this.entityReferenceSerializer.serialize(reference), "UTF-8"); > } catch (UnsupportedEncodingException e) { > // Not supporting UTF-8 as a valid encoding for some reasons. We consider XWiki cannot work > // without that encoding. > - throw new RuntimeException("Failed to URL encode [" + this.entityReferenceSerializer.serialize( > - reference) + "] using UTF-8.", e); > + throw new RuntimeException("Failed to URL encode [" > + + this.entityReferenceSerializer.serialize(reference) + "] using UTF-8.", e); > } > } > } > - > + > return this.documentAccessBridge.getURL(documentName, "create", modifiedQueryString, anchor); > } > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -22,9 +22,9 @@ > import java.util.Properties; > > import org.jmock.Expectations; > -import org.xwiki.configuration.ConfigurationSource; > import org.junit.Assert; > import org.junit.Test; > +import org.xwiki.configuration.ConfigurationSource; > import org.xwiki.test.AbstractMockingComponentTestCase; > import org.xwiki.test.annotation.MockingRequirement; > > @@ -43,10 +43,13 @@ > public void testGetLinkLabelFormat() throws Exception > { > final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); > - getMockery().checking(new Expectations() {{ > - allowing(source).getProperty("rendering.linkLabelFormat", "%p"); > + getMockery().checking(new Expectations() > + { > + { > + allowing(source).getProperty("rendering.linkLabelFormat", "%p"); > will(returnValue("%p")); > - }}); > + } > + }); > > Assert.assertEquals("%p", this.configuration.getLinkLabelFormat()); > } > @@ -55,12 +58,60 @@ > public void testGetMacroCategories() throws Exception > { > final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); > - getMockery().checking(new Expectations() {{ > - allowing(source).getProperty("rendering.macroCategories", Properties.class); > + getMockery().checking(new Expectations() > + { > + { > + allowing(source).getProperty("rendering.macroCategories", Properties.class); > will(returnValue(new Properties())); > - }}); > + } > + }); > > Assert.assertNotNull(this.configuration.getMacroCategories()); > Assert.assertEquals(0, this.configuration.getMacroCategories().size()); > } > + > + @Test > + public void testGetImageWidthLimit() throws Exception > + { > + final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); > + getMockery().checking(new Expectations() > + { > + { > + allowing(source).getProperty("rendering.imageWidthLimit", -1); > + will(returnValue(100)); > + } > + }); > + > + Assert.assertEquals(100, this.configuration.getImageWidthLimit()); > + } > + > + @Test > + public void testGetImageHeightLimit() throws Exception > + { > + final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); > + getMockery().checking(new Expectations() > + { > + { > + allowing(source).getProperty("rendering.imageHeightLimit", -1); > + will(returnValue(150)); > + } > + }); > + > + Assert.assertEquals(150, this.configuration.getImageHeightLimit()); > + } > + > + @Test > + public void testIsIncludeImageDimensionsInImageURL() throws Exception > + { > + final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); > + getMockery().checking(new Expectations() > + { > + { > + allowing(source).getProperty("rendering.includeImageDimensionsInImageURL", true); > + will(returnValue(false)); > + } > + }); > + > + Assert.assertFalse(this.configuration.isIncludeImageDimensionsInImageURL()); > + } > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -19,12 +19,20 @@ > */ > package org.xwiki.rendering.internal.wiki; > > +import java.util.Collections; > +import java.util.HashMap; > +import java.util.Map; > + > +import junit.framework.Assert; > + > import org.jmock.Expectations; > -import org.jmock.Mockery; > +import org.junit.Test; > import org.xwiki.bridge.DocumentAccessBridge; > import org.xwiki.model.reference.DocumentReference; > -import org.xwiki.component.util.ReflectionUtils; > import org.xwiki.model.reference.EntityReferenceSerializer; > +import org.xwiki.rendering.configuration.RenderingConfiguration; > +import org.xwiki.test.AbstractMockingComponentTestCase; > +import org.xwiki.test.annotation.MockingRequirement; > > /** > * Unit tests for {@link XWikiWikiModel}. > @@ -32,31 +40,321 @@ > * @version $Id$ > * @since 2.0M1 > */ > -public class XWikiWikiModelTest > +public class XWikiWikiModelTest extends AbstractMockingComponentTestCase > { > - @org.junit.Test > + @MockingRequirement > + private XWikiWikiModel wikiModel; > + > + @Test > public void testGetDocumentEditURLWhenNoQueryStringSpecified() throws Exception > { > - Mockery mockery = new Mockery(); > - XWikiWikiModel wikiModel = new XWikiWikiModel(); > + final EntityReferenceSerializer< ? > entityReferenceSerializer = > + getComponentManager().lookup(EntityReferenceSerializer.class); > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final DocumentReference documentReference = new DocumentReference("wiki", "Space", "Page"); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getCurrentDocumentReference(); > + will(returnValue(documentReference)); > + oneOf(entityReferenceSerializer).serialize(documentReference); > + will(returnValue("wiki:Space.Page\u20AC")); > > - final EntityReferenceSerializer mockEntityReferenceSerializer = mockery.mock(EntityReferenceSerializer.class); > - ReflectionUtils.setFieldValue(wikiModel, "entityReferenceSerializer", mockEntityReferenceSerializer); > + // The test is here: we verify that getURL is called with the query string already encoded since > + // getURL() doesn't encode it. > + oneOf(documentAccessBridge).getURL("Space.Page\u20AC", "create", "parent=wiki%3ASpace.Page%E2%82%AC", > + "anchor"); > + } > + }); > > - final DocumentAccessBridge mockDocumentAccessBridge = mockery.mock(DocumentAccessBridge.class); > - ReflectionUtils.setFieldValue(wikiModel, "documentAccessBridge", mockDocumentAccessBridge); > + wikiModel.getDocumentEditURL("Space.Page\u20AC", "anchor", null); > + } > > - final DocumentReference docReference = new DocumentReference("wiki", "Space", "Page"); > - mockery.checking(new Expectations() {{ > - oneOf(mockDocumentAccessBridge).getCurrentDocumentReference(); will(returnValue(docReference)); > - oneOf(mockEntityReferenceSerializer).serialize(docReference); will(returnValue("wiki:Space.Page\u20AC")); > + /** > + * Tests that the proper image URL is generated when both the width and the height image parameters are specified. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothWidthAndHeightAttributesAreSpecified() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("width", "100px"); > + parameters.put("height", "50"); > + Assert.assertEquals("?width=100&height=50", wikiModel.getImageURL("", "", parameters)); > + } > > - // The test is here: we verify that getURL is called with the query string already encoded since getURL() > - // doesn't encode it. > - oneOf(mockDocumentAccessBridge).getURL("Space.Page\u20AC", "create", "parent=wiki%3ASpace.Page%E2%82%AC", > - "anchor"); > - }}); > + /** > + * Tests that the proper image URL is generated when both the width and the height image parameters are specified > + * but including them in the image URL is forbidden from the rendering configuration. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenIncludingImageDimensionsIsForbidden() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(false)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("width", "101px"); > + parameters.put("height", "55px"); > + Assert.assertEquals("", wikiModel.getImageURL("", "", parameters)); > + } > > - wikiModel.getDocumentEditURL("Space.Page\u20AC", "anchor", null); > + /** > + * Tests that the proper image URL is generated when both the width and the height CSS properties are specified. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothWidthAndHeightCSSPropertiesAreSpecified() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("style", "border: 1px; height: 30px; margin-top: 2em; width: 70px"); > + Assert.assertEquals("?width=70&height=30", wikiModel.getImageURL("", "", parameters)); > } > + > + /** > + * Tests that the proper image URL is generated when only the width image parameter is specified. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenOnlyWidthAttributeIsSpecified() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("width", "150"); > + parameters.put("height", "30%"); > + Assert.assertEquals("?width=150", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when only the height CSS property is specified. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenOnlyHeightCSSPropertyIsSpecified() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("style", "width: 5cm; height: 80px"); > + Assert.assertEquals("?height=80", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when both the width and the height are unspecified and image size is > + * limited in the configuration. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothWidthAndHeightAreUnspecifiedAndImageSizeIsLimited() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + oneOf(configuration).getImageWidthLimit(); > + will(returnValue(200)); > + oneOf(configuration).getImageHeightLimit(); > + will(returnValue(170)); > + } > + }); > + Map<String, String> parameters = Collections.emptyMap(); > + Assert.assertEquals("?width=200&height=170&keepAspectRatio=true", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when both the width and the height are unspecified and only the > + * image width is limited in the configuration. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothWidthAndHeightAreUnspecifiedAndOnlyImageWidthIsLimited() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + oneOf(configuration).getImageWidthLimit(); > + will(returnValue(25)); > + oneOf(configuration).getImageHeightLimit(); > + will(returnValue(-1)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("width", "45%"); > + parameters.put("style", "height:10em"); > + Assert.assertEquals("?width=25", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when both the width and the height are unspecified and the image > + * size is not limited in the configuration. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothWidthAndHeightAreUnspecifiedAndImageSizeIsNotLimited() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + oneOf(configuration).getImageWidthLimit(); > + will(returnValue(-1)); > + oneOf(configuration).getImageHeightLimit(); > + will(returnValue(-1)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("style", "bad CSS declaration"); > + Assert.assertEquals("", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when the attachment URL has a fragment identifier. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenAttachmentURLHasFragmentIdentifier() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("test#fragment")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("width", "23"); > + Assert.assertEquals("test?width=23#fragment", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when the attachment URL has a query string and a fragment > + * identifier. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenAttachmentURLHasQueryStringAndFragmentIdentifier() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("test?param=value#fragment")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("height", "17"); > + Assert.assertEquals("test?param=value&height=17#fragment", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when both the style and the dimension parameters are specified. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothStyleAndDimensionParametersAreSpecified() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("height", "46"); > + parameters.put("width", "101px"); > + parameters.put("style", "width: 20%; height:75px"); > + // Note that the style parameter take precedence over the dimension parameters and the width is actually 20% but > + // we can't use it for resizing the image on the server side so it's omitted from the query string. > + Assert.assertEquals("?height=75", wikiModel.getImageURL("", "", parameters)); > + } > } > > _______________________________________________ > notifications mailing list > notifications(a)xwiki.org > http://lists.xwiki.org/mailman/listinfo/notifications

15 years

4
10
0 0

Re: [xwiki-devs] [xwiki-notifications] r31066 - platform/core/trunk/xwiki-classloader/src/main/java/org/xwiki/classloader/internal

by Sergiu Dumitriu

On 09/13/2010 01:34 PM, tmortagne (SVN) wrote: > Author: tmortagne > Date: 2010-09-13 13:34:11 +0200 (Mon, 13 Sep 2010) > New Revision: 31066 > > Modified: > platform/core/trunk/xwiki-classloader/src/main/java/org/xwiki/classloader/internal/ResourceLoader.java > Log: > XWIKI-5482: URIClassLoader#findResources can return infinite Enumeration > > Modified: platform/core/trunk/xwiki-classloader/src/main/java/org/xwiki/classloader/internal/ResourceLoader.java > =================================================================== > --- platform/core/trunk/xwiki-classloader/src/main/java/org/xwiki/classloader/internal/ResourceLoader.java 2010-09-13 09:06:31 UTC (rev 31065) > +++ platform/core/trunk/xwiki-classloader/src/main/java/org/xwiki/classloader/internal/ResourceLoader.java 2010-09-13 11:34:11 UTC (rev 31066) > @@ -803,7 +803,11 @@ > fetchNext(); > if (next == null) Codestyle, there should be {} around the next line. > throw new NoSuchElementException(); > - return next; > + > + Object nextElement = next; > + next = null; > + > + return nextElement; > } > > private void fetchNext() -- Sergiu Dumitriu http://purl.org/net/sergiu/

15 years

2
1
0 0

Re: [xwiki-devs] [xwiki-notifications] r31053 - in platform/core/trunk/xwiki-rendering: xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki xwiki-rendering-xwiki xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration xwiki-rendering-xwiki/src/main/java/org/x

by Vincent Massol

Hi Marius, I think there are some changes we need to discuss in this commit: - new config api methods - new wiki model api methods - new deps introduced in the rendering module (makes it harder to use without maven) These 2 are important as they are important public apis. The WikiModel one is the most important and we need to be 100% sure we want to add a new methods. There must absolutely be the minimal number of methods in this class. Also we're going to break people using the rendering module outside of xwiki (guys from eXo for example, so we need to be careful). I haven't thought too much about it yet but my first feeling is that the methods added look quite specific. I'd like to have Thomas's opinion on this since he's the one who's the most knowledgeable about the rendering module. Thanks -Vincent On Sep 12, 2010, at 1:30 PM, mflorea (SVN) wrote: > Author: mflorea > Date: 2010-09-12 13:30:15 +0200 (Sun, 12 Sep 2010) > New Revision: 31053 > > Modified: > platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml > platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java > Log: > XWIKI-5479: Include image dimensions in the image URL for server side image resizing > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -50,7 +50,7 @@ > * </ul> > * Note that if the page title is empty or not defined then it defaults to %p. This is also the case > * if the title cannot be retrieved for the document. > - * > + * > * The default is "%p". Some examples: "%s.%p", "%w:%s.%p". > * > * @return the format to use to display link labels when the user hasn't specified a label > @@ -63,4 +63,42 @@ > * descriptor (ie defined by the macro author) will be used > */ > Properties getMacroCategories(); > + > + /** > + * @return {@code true} to include the image dimensions extracted from the image parameters in the image URL, > + * {@code false} otherwise; when image dimensions are included in the image URL the image can be resized > + * on the server side before being downloaded. > + * @since 2.5M2 > + */ > + boolean isIncludeImageDimensionsInImageURL(); > + > + /** > + * One way to improve page load speed is to resize images on the server side just before rendering the page. The > + * rendering module can use the image width provided by the user to scale the image. When the user doesn't specify > + * the image width the rendering module can limit the width of the image based on this configuration parameter. > + * <p> > + * The default value is {@code -1} which means image width is not limited by default. Use a value greater than 0 to > + * limit the image width (pixels). Note that the aspect ratio is kept even when both the width and the height of the > + * image are limited. > + * > + * @return the maximum image width when there's no user supplied width > + * @see #isIncludeImageDimensionsInImageURL() > + * @since 2.5M2 > + */ > + int getImageWidthLimit(); > + > + /** > + * One way to improve page load speed is to resize images on the server side just before rendering the page. The > + * rendering module can use the image height provided by the user to scale the image. When the user doesn't specify > + * the image height the rendering module can limit the height of the image based on this configuration parameter. > + * <p> > + * The default value is {@code -1} which means image height is not limited by default. Use a value greater than 0 to > + * limit the image height (pixels). Note that the aspect ratio is kept even when both the width and the height of > + * the image are limited. > + * > + * @return the maximum image height when there's no user supplied height > + * @see #isIncludeImageDimensionsInImageURL() > + * @since 2.5M2 > + */ > + int getImageHeightLimit(); > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -45,6 +45,21 @@ > private Properties macroCategories = new Properties(); > > /** > + * @see #getImageWidthLimit() > + */ > + private int imageWidthLimit = -1; > + > + /** > + * @see #getImageHeightLimit() > + */ > + private int imageHeightLimit = -1; > + > + /** > + * @see #isIncludeImageDimensionsInImageURL() > + */ > + private boolean includeImageDimensionsInImageURL = true; > + > + /** > * {@inheritDoc} > * > * @see org.xwiki.rendering.configuration.RenderingConfiguration#getLinkLabelFormat() > @@ -84,4 +99,66 @@ > // to work even without a configuration store. > this.macroCategories.setProperty(macroId.toString(), category); > } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageWidthLimit() > + */ > + public int getImageWidthLimit() > + { > + return this.imageWidthLimit; > + } > + > + /** > + * @param imageWidthLimit the maximum image width when there's no user supplied width > + */ > + public void setImageWidthLimit(int imageWidthLimit) > + { > + // This method is useful for those using the XWiki Rendering in standalone mode since it allows the rendering > + // to work even without a configuration store. > + this.imageWidthLimit = imageWidthLimit; > + } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageHeightLimit() > + */ > + public int getImageHeightLimit() > + { > + return this.imageHeightLimit; > + } > + > + /** > + * @param imageHeightLimit the maximum image height when there's no user supplied height > + */ > + public void setImageHeightLimit(int imageHeightLimit) > + { > + // This method is useful for those using the XWiki Rendering in standalone mode since it allows the rendering > + // to work even without a configuration store. > + this.imageHeightLimit = imageHeightLimit; > + } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#isIncludeImageDimensionsInImageURL() > + */ > + public boolean isIncludeImageDimensionsInImageURL() > + { > + return this.includeImageDimensionsInImageURL; > + } > + > + /** > + * @param includeImageDimensionsInImageURL {@code true} to include image dimensions extracted from the image > + * parameters in the image URL so that the image can be resized on the server side before being > + * downloaded, {@code false} otherwise > + */ > + public void setIncludeImageDimensionsInImageURL(boolean includeImageDimensionsInImageURL) > + { > + // This method is useful for those using the XWiki Rendering in standalone mode since it allows the rendering > + // to work even without a configuration store. > + this.includeImageDimensionsInImageURL = includeImageDimensionsInImageURL; > + } > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -42,10 +42,9 @@ > * Default implementation for rendering images as XHTML. We handle both cases: > * <ul> > * <li>when inside a wiki (ie when an implementation of {@link WikiModel} is provided.</li> > - * <li>when outside of a wiki. In this case we only handle external images and document images don't > - * display anything.</li> > + * <li>when outside of a wiki. In this case we only handle external images and document images don't display anything.</li> > * </ul> > - * > + * > * @version $Id$ > * @since 2.0M3 > */ > @@ -68,6 +67,7 @@ > > /** > * {@inheritDoc} > + * > * @see Initializable#initialize() > */ > public void initialize() throws InitializationException > @@ -83,7 +83,7 @@ > > /** > * {@inheritDoc} > - * > + * > * @see XHTMLImageRenderer#setXHTMLWikiPrinter(XHTMLWikiPrinter) > */ > public void setXHTMLWikiPrinter(XHTMLWikiPrinter printer) > @@ -103,7 +103,7 @@ > > /** > * {@inheritDoc} > - * > + * > * @see XHTMLImageRenderer#onImage(Image, boolean, Map) > */ > public void onImage(Image image, boolean isFreeStandingURI, Map<String, String> parameters) > @@ -118,8 +118,9 @@ > // that would not honor this contract. > if (this.wikiModel != null) { > DocumentImage documentImage = DocumentImage.class.cast(image); > - imageURL = this.wikiModel.getAttachmentURL(documentImage.getDocumentName(), > - documentImage.getAttachmentName()); > + imageURL = > + this.wikiModel.getImageURL(documentImage.getDocumentName(), documentImage.getAttachmentName(), > + parameters); > } else { > throw new RuntimeException("Invalid Image type. In non wiki mode, all image types must be URL images."); > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -19,13 +19,15 @@ > */ > package org.xwiki.rendering.wiki; > > +import java.util.Map; > + > import org.xwiki.component.annotation.ComponentRole; > > /** > * Bridge between the Rendering module and a Wiki Model. Contains wiki APIs required by Rendering classes such as > - * Renderers. For example the XHTML Link Renderer needs to know if a wiki document exists in order to know how to > - * generate the HTML (in order to display a question mark for non existing documents) and it also needs to get the > - * URL pointing the wiki document. > + * Renderers. For example the XHTML Link Renderer needs to know if a wiki document exists in order to know how to > + * generate the HTML (in order to display a question mark for non existing documents) and it also needs to get the URL > + * pointing the wiki document. > * > * @version $Id$ > * @since 2.0M1 > @@ -34,36 +36,48 @@ > public interface WikiModel > { > /** > - * @param documentName the document name of the document containing the attachment. The syntax used depends on > - * the underlying wiki system used. For example for XWiki a valid documentName would be > - * {@code wiki:Space.Page} > + * @param documentName the document name of the document containing the attachment. The syntax used depends on the > + * underlying wiki system used. For example for XWiki a valid documentName would be > + * {@code wiki:Space.Page} > * @param attachmentName the name of the attachment in the passed document wikip page > * @return the URL to the attachment > */ > String getAttachmentURL(String documentName, String attachmentName); > > /** > - * @param documentName the document name as a String. The syntax used depends on > - * the underlying wiki system used. For example for XWiki a valid documentName would be > - * {@code wiki:Space.Page} > + * Generate image specific URL. The difference with {@link #getAttachmentURL(String, String)} is that in some > + * implementation we want to make a distinction between displayed image and a simple link targeting an attachment > + * file. > + * > + * @param documentName the document name of the document containing the attachment. The syntax used depends on the > + * underlying wiki system used. For example for XWiki a valid documentName would be > + * {@code wiki:Space.Page} > + * @param attachmentName the name of the attachment in the passed document wiki page > + * @param parameters custom parameters > + * @return the URL to the image > + * @since 2.5M2 > + */ > + String getImageURL(String documentName, String attachmentName, Map<String, String> parameters); > + > + /** > + * @param documentName the document name as a String. The syntax used depends on the underlying wiki system used. > + * For example for XWiki a valid documentName would be {@code wiki:Space.Page} > * @return true if the document exists and can be viewed or false otherwise > */ > boolean isDocumentAvailable(String documentName); > > /** > - * @param documentName the document name as a String. The syntax used depends on > - * the underlying wiki system used. For example for XWiki a valid documentName would be > - * {@code wiki:Space.Page} > + * @param documentName the document name as a String. The syntax used depends on the underlying wiki system used. > + * For example for XWiki a valid documentName would be {@code wiki:Space.Page} > * @param anchor an anchor pointing to some place inside the document or null > * @param queryString a query string specifying some parameters or null > * @return the URL to view the specified wiki document > */ > String getDocumentViewURL(String documentName, String anchor, String queryString); > - > + > /** > - * @param documentName the document name as a String. The syntax used depends on > - * the underlying wiki system used. For example for XWiki a valid documentName would be > - * {@code wiki:Space.Page} > + * @param documentName the document name as a String. The syntax used depends on the underlying wiki system used. > + * For example for XWiki a valid documentName would be {@code wiki:Space.Page} > * @param anchor an anchor pointing to some place inside the document or null > * @param queryString a query string specifying some parameters or null > * @return the URL to edit the specified wiki document > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml 2010-09-12 11:30:15 UTC (rev 31053) > @@ -54,6 +54,12 @@ > <artifactId>xwiki-core-script</artifactId> > <version>${pom.version}</version> > </dependency> > + <dependency> > +  > + <groupId>net.sourceforge.cssparser</groupId> > + <artifactId>cssparser</artifactId> > + <version>0.9.5</version> > + </dependency> >  > <dependency> > <groupId>org.xwiki.platform</groupId> > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -19,13 +19,13 @@ > */ > package org.xwiki.rendering.internal.configuration; > > +import java.util.Properties; > + > import org.xwiki.component.annotation.Component; > import org.xwiki.component.annotation.Requirement; > import org.xwiki.configuration.ConfigurationSource; > import org.xwiki.rendering.configuration.RenderingConfiguration; > > -import java.util.Properties; > - > /** > * All configuration options for the rendering subsystem. > * > @@ -46,7 +46,7 @@ > private static final String DEFAULT_LINK_LABEL_FORMAT = "%p"; > > /** > - * Defines from where to read the rendering configuration data. > + * Defines from where to read the rendering configuration data. > */ > @Requirement > private ConfigurationSource configuration; > @@ -58,16 +58,46 @@ > */ > public String getLinkLabelFormat() > { > - return this.configuration.getProperty(PREFIX + "linkLabelFormat", DEFAULT_LINK_LABEL_FORMAT); > + return this.configuration.getProperty(PREFIX + "linkLabelFormat", DEFAULT_LINK_LABEL_FORMAT); > } > > /** > * {@inheritDoc} > - * > + * > * @see org.xwiki.rendering.configuration.RenderingConfiguration#getMacroCategories() > */ > public Properties getMacroCategories() > { > return this.configuration.getProperty(PREFIX + "macroCategories", Properties.class); > } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageWidthLimit() > + */ > + public int getImageWidthLimit() > + { > + return this.configuration.getProperty(PREFIX + "imageWidthLimit", -1); > + } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageHeightLimit() > + */ > + public int getImageHeightLimit() > + { > + return this.configuration.getProperty(PREFIX + "imageHeightLimit", -1); > + } > + > + /** > + * {@inheritDoc} > + * > + * @see org.xwiki.rendering.configuration.RenderingConfiguration#isIncludeImageDimensionsInImageURL() > + */ > + public boolean isIncludeImageDimensionsInImageURL() > + { > + return this.configuration.getProperty(PREFIX + "includeImageDimensionsInImageURL", true); > + } > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -19,17 +19,25 @@ > */ > package org.xwiki.rendering.internal.wiki; > > +import java.io.IOException; > +import java.io.StringReader; > import java.io.UnsupportedEncodingException; > import java.net.URLEncoder; > +import java.util.Map; > > import org.apache.commons.lang.StringUtils; > +import org.w3c.css.sac.InputSource; > +import org.w3c.dom.css.CSSStyleDeclaration; > import org.xwiki.bridge.DocumentAccessBridge; > -import org.xwiki.model.reference.DocumentReference; > import org.xwiki.component.annotation.Component; > import org.xwiki.component.annotation.Requirement; > +import org.xwiki.model.reference.DocumentReference; > import org.xwiki.model.reference.EntityReferenceSerializer; > +import org.xwiki.rendering.configuration.RenderingConfiguration; > import org.xwiki.rendering.wiki.WikiModel; > > +import com.steadystate.css.parser.CSSOMParser; > + > /** > * Implementation using the Document Access Bridge ({@link DocumentAccessBridge}). > * > @@ -39,14 +47,47 @@ > @Component > public class XWikiWikiModel implements WikiModel > { > + /** > + * The suffix used to mark an amount of pixels. > + */ > + private static final String PIXELS = "px"; > + > + /** > + * The name of the {@code width} image parameter. > + */ > + private static final String WIDTH = "width"; > + > + /** > + * The name of the {@code height} image parameter. > + */ > + private static final String HEIGHT = "height"; > + > + /** > + * The component used to access configuration parameters. > + */ > @Requirement > + private RenderingConfiguration renderingConfiguration; > + > + /** > + * The component used to access the underlying XWiki model. > + */ > + @Requirement > private DocumentAccessBridge documentAccessBridge; > > + /** > + * The component used to serialize entity references to strings. > + */ > @Requirement > private EntityReferenceSerializer<String> entityReferenceSerializer; > > /** > + * The object used to parse the CSS from the image style parameter. > + */ > + private final CSSOMParser cssParser = new CSSOMParser(); > + > + /** > * {@inheritDoc} > + * > * @see WikiModel#getAttachmentURL(String, String) > */ > public String getAttachmentURL(String documentName, String attachmentName) > @@ -55,7 +96,112 @@ > } > > /** > + * Extracts the specified image dimension from the image parameters. > + * > + * @param dimension either {@code width} or {@code height} > + * @param imageParameters the image parameters; may include the {@code width}, {@code height} and {@code style} > + * parameters > + * @return the value of the passed dimension if it is specified in the image parameters, {@code null} otherwise > + */ > + private String getImageDimension(String dimension, Map<String, String> imageParameters) > + { > + // Check first if the style parameter contains information about the given dimension. In-line style has priority > + // over the dimension parameters. > + String value = null; > + String style = imageParameters.get("style"); > + if (StringUtils.isNotBlank(style)) { > + try { > + CSSStyleDeclaration sd = cssParser.parseStyleDeclaration(new InputSource(new StringReader(style))); > + value = sd.getPropertyValue(dimension); > + } catch (IOException e) { > + // Ignore the style parameter. > + } > + } > + if (StringUtils.isBlank(value)) { > + // Fall back on the value of the dimension parameter. > + value = imageParameters.get(dimension); > + } > + return value; > + } > + > + /** > + * Creates the query string that can be added to an image URL to resize the image on the server side. > + * > + * @param imageParameters image parameters, including width and height then they are specified > + * @return the query string to be added to an image URL in order to resize the image on the server side > + */ > + private StringBuilder getImageURLQueryString(Map<String, String> imageParameters) > + { > + String width = StringUtils.chomp(getImageDimension(WIDTH, imageParameters), PIXELS); > + String height = StringUtils.chomp(getImageDimension(HEIGHT, imageParameters), PIXELS); > + boolean useHeight = StringUtils.isNotEmpty(height) && StringUtils.isNumeric(height); > + StringBuilder queryString = new StringBuilder(); > + if (StringUtils.isEmpty(width) || !StringUtils.isNumeric(width)) { > + // Width is unspecified or is not measured in pixels. > + if (useHeight) { > + // Height is specified in pixels. > + queryString.append('&').append(HEIGHT).append('=').append(height); > + } else { > + // If image width and height are unspecified or if they are not expressed in pixels then limit the image > + // size to best fit the rectangle specified in the configuration (keeping aspect ratio). > + int widthLimit = renderingConfiguration.getImageWidthLimit(); > + if (widthLimit > 0) { > + queryString.append('&').append(WIDTH).append('=').append(widthLimit); > + } > + int heightLimit = renderingConfiguration.getImageHeightLimit(); > + if (heightLimit > 0) { > + queryString.append('&').append(HEIGHT).append('=').append(heightLimit); > + } > + if (widthLimit > 0 && heightLimit > 0) { > + queryString.append("&keepAspectRatio=").append(true); > + } > + } > + } else { > + // Width is specified in pixels. > + queryString.append('&').append(WIDTH).append('=').append(width); > + if (useHeight) { > + // Height is specified in pixels. > + queryString.append('&').append(HEIGHT).append('=').append(height); > + } > + } > + return queryString; > + } > + > + /** > * {@inheritDoc} > + * > + * @see org.xwiki.rendering.wiki.WikiModel#getImageURL(java.lang.String, java.lang.String, java.util.Map) > + */ > + public String getImageURL(String documentName, String attachmentName, Map<String, String> parameters) > + { > + String url = getAttachmentURL(documentName, attachmentName); > + if (!renderingConfiguration.isIncludeImageDimensionsInImageURL()) { > + return url; > + } > + > + StringBuilder queryString = getImageURLQueryString(parameters); > + if (queryString.length() == 0) { > + return url; > + } > + > + // Determine the insertion point. > + int insertionPoint = url.lastIndexOf('#'); > + if (insertionPoint < 0) { > + // No fragment identifier. > + insertionPoint = url.length(); > + } > + if (url.lastIndexOf('?', insertionPoint) < 0) { > + // No query string. > + queryString.setCharAt(0, '?'); > + } > + > + // Insert the query string. > + return new StringBuilder(url).insert(insertionPoint, queryString).toString(); > + } > + > + /** > + * {@inheritDoc} > + * > * @see WikiModel#isDocumentAvailable(String) > */ > public boolean isDocumentAvailable(String documentName) > @@ -65,6 +211,7 @@ > > /** > * {@inheritDoc} > + * > * @see WikiModel#getDocumentViewURL(String, String, String) > */ > public String getDocumentViewURL(String documentName, String anchor, String queryString) > @@ -74,6 +221,7 @@ > > /** > * {@inheritDoc} > + * > * @see WikiModel#getDocumentEditURL(String, String, String) > */ > public String getDocumentEditURL(String documentName, String anchor, String queryString) > @@ -89,17 +237,17 @@ > // See http://www.w3.org/TR/html40/appendix/notes.html#non-ascii-chars > // TODO: Once the xwiki-url module is usable, refactor this code to use it and remove the need to > // perform explicit encoding here. > - modifiedQueryString = > + modifiedQueryString = > "parent=" + URLEncoder.encode(this.entityReferenceSerializer.serialize(reference), "UTF-8"); > } catch (UnsupportedEncodingException e) { > // Not supporting UTF-8 as a valid encoding for some reasons. We consider XWiki cannot work > // without that encoding. > - throw new RuntimeException("Failed to URL encode [" + this.entityReferenceSerializer.serialize( > - reference) + "] using UTF-8.", e); > + throw new RuntimeException("Failed to URL encode [" > + + this.entityReferenceSerializer.serialize(reference) + "] using UTF-8.", e); > } > } > } > - > + > return this.documentAccessBridge.getURL(documentName, "create", modifiedQueryString, anchor); > } > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -22,9 +22,9 @@ > import java.util.Properties; > > import org.jmock.Expectations; > -import org.xwiki.configuration.ConfigurationSource; > import org.junit.Assert; > import org.junit.Test; > +import org.xwiki.configuration.ConfigurationSource; > import org.xwiki.test.AbstractMockingComponentTestCase; > import org.xwiki.test.annotation.MockingRequirement; > > @@ -43,10 +43,13 @@ > public void testGetLinkLabelFormat() throws Exception > { > final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); > - getMockery().checking(new Expectations() {{ > - allowing(source).getProperty("rendering.linkLabelFormat", "%p"); > + getMockery().checking(new Expectations() > + { > + { > + allowing(source).getProperty("rendering.linkLabelFormat", "%p"); > will(returnValue("%p")); > - }}); > + } > + }); > > Assert.assertEquals("%p", this.configuration.getLinkLabelFormat()); > } > @@ -55,12 +58,60 @@ > public void testGetMacroCategories() throws Exception > { > final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); > - getMockery().checking(new Expectations() {{ > - allowing(source).getProperty("rendering.macroCategories", Properties.class); > + getMockery().checking(new Expectations() > + { > + { > + allowing(source).getProperty("rendering.macroCategories", Properties.class); > will(returnValue(new Properties())); > - }}); > + } > + }); > > Assert.assertNotNull(this.configuration.getMacroCategories()); > Assert.assertEquals(0, this.configuration.getMacroCategories().size()); > } > + > + @Test > + public void testGetImageWidthLimit() throws Exception > + { > + final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); > + getMockery().checking(new Expectations() > + { > + { > + allowing(source).getProperty("rendering.imageWidthLimit", -1); > + will(returnValue(100)); > + } > + }); > + > + Assert.assertEquals(100, this.configuration.getImageWidthLimit()); > + } > + > + @Test > + public void testGetImageHeightLimit() throws Exception > + { > + final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); > + getMockery().checking(new Expectations() > + { > + { > + allowing(source).getProperty("rendering.imageHeightLimit", -1); > + will(returnValue(150)); > + } > + }); > + > + Assert.assertEquals(150, this.configuration.getImageHeightLimit()); > + } > + > + @Test > + public void testIsIncludeImageDimensionsInImageURL() throws Exception > + { > + final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); > + getMockery().checking(new Expectations() > + { > + { > + allowing(source).getProperty("rendering.includeImageDimensionsInImageURL", true); > + will(returnValue(false)); > + } > + }); > + > + Assert.assertFalse(this.configuration.isIncludeImageDimensionsInImageURL()); > + } > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java 2010-09-11 17:15:46 UTC (rev 31052) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java 2010-09-12 11:30:15 UTC (rev 31053) > @@ -19,12 +19,20 @@ > */ > package org.xwiki.rendering.internal.wiki; > > +import java.util.Collections; > +import java.util.HashMap; > +import java.util.Map; > + > +import junit.framework.Assert; > + > import org.jmock.Expectations; > -import org.jmock.Mockery; > +import org.junit.Test; > import org.xwiki.bridge.DocumentAccessBridge; > import org.xwiki.model.reference.DocumentReference; > -import org.xwiki.component.util.ReflectionUtils; > import org.xwiki.model.reference.EntityReferenceSerializer; > +import org.xwiki.rendering.configuration.RenderingConfiguration; > +import org.xwiki.test.AbstractMockingComponentTestCase; > +import org.xwiki.test.annotation.MockingRequirement; > > /** > * Unit tests for {@link XWikiWikiModel}. > @@ -32,31 +40,321 @@ > * @version $Id$ > * @since 2.0M1 > */ > -public class XWikiWikiModelTest > +public class XWikiWikiModelTest extends AbstractMockingComponentTestCase > { > - @org.junit.Test > + @MockingRequirement > + private XWikiWikiModel wikiModel; > + > + @Test > public void testGetDocumentEditURLWhenNoQueryStringSpecified() throws Exception > { > - Mockery mockery = new Mockery(); > - XWikiWikiModel wikiModel = new XWikiWikiModel(); > + final EntityReferenceSerializer< ? > entityReferenceSerializer = > + getComponentManager().lookup(EntityReferenceSerializer.class); > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final DocumentReference documentReference = new DocumentReference("wiki", "Space", "Page"); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getCurrentDocumentReference(); > + will(returnValue(documentReference)); > + oneOf(entityReferenceSerializer).serialize(documentReference); > + will(returnValue("wiki:Space.Page\u20AC")); > > - final EntityReferenceSerializer mockEntityReferenceSerializer = mockery.mock(EntityReferenceSerializer.class); > - ReflectionUtils.setFieldValue(wikiModel, "entityReferenceSerializer", mockEntityReferenceSerializer); > + // The test is here: we verify that getURL is called with the query string already encoded since > + // getURL() doesn't encode it. > + oneOf(documentAccessBridge).getURL("Space.Page\u20AC", "create", "parent=wiki%3ASpace.Page%E2%82%AC", > + "anchor"); > + } > + }); > > - final DocumentAccessBridge mockDocumentAccessBridge = mockery.mock(DocumentAccessBridge.class); > - ReflectionUtils.setFieldValue(wikiModel, "documentAccessBridge", mockDocumentAccessBridge); > + wikiModel.getDocumentEditURL("Space.Page\u20AC", "anchor", null); > + } > > - final DocumentReference docReference = new DocumentReference("wiki", "Space", "Page"); > - mockery.checking(new Expectations() {{ > - oneOf(mockDocumentAccessBridge).getCurrentDocumentReference(); will(returnValue(docReference)); > - oneOf(mockEntityReferenceSerializer).serialize(docReference); will(returnValue("wiki:Space.Page\u20AC")); > + /** > + * Tests that the proper image URL is generated when both the width and the height image parameters are specified. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothWidthAndHeightAttributesAreSpecified() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("width", "100px"); > + parameters.put("height", "50"); > + Assert.assertEquals("?width=100&height=50", wikiModel.getImageURL("", "", parameters)); > + } > > - // The test is here: we verify that getURL is called with the query string already encoded since getURL() > - // doesn't encode it. > - oneOf(mockDocumentAccessBridge).getURL("Space.Page\u20AC", "create", "parent=wiki%3ASpace.Page%E2%82%AC", > - "anchor"); > - }}); > + /** > + * Tests that the proper image URL is generated when both the width and the height image parameters are specified > + * but including them in the image URL is forbidden from the rendering configuration. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenIncludingImageDimensionsIsForbidden() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(false)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("width", "101px"); > + parameters.put("height", "55px"); > + Assert.assertEquals("", wikiModel.getImageURL("", "", parameters)); > + } > > - wikiModel.getDocumentEditURL("Space.Page\u20AC", "anchor", null); > + /** > + * Tests that the proper image URL is generated when both the width and the height CSS properties are specified. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothWidthAndHeightCSSPropertiesAreSpecified() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("style", "border: 1px; height: 30px; margin-top: 2em; width: 70px"); > + Assert.assertEquals("?width=70&height=30", wikiModel.getImageURL("", "", parameters)); > } > + > + /** > + * Tests that the proper image URL is generated when only the width image parameter is specified. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenOnlyWidthAttributeIsSpecified() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("width", "150"); > + parameters.put("height", "30%"); > + Assert.assertEquals("?width=150", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when only the height CSS property is specified. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenOnlyHeightCSSPropertyIsSpecified() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("style", "width: 5cm; height: 80px"); > + Assert.assertEquals("?height=80", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when both the width and the height are unspecified and image size is > + * limited in the configuration. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothWidthAndHeightAreUnspecifiedAndImageSizeIsLimited() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + oneOf(configuration).getImageWidthLimit(); > + will(returnValue(200)); > + oneOf(configuration).getImageHeightLimit(); > + will(returnValue(170)); > + } > + }); > + Map<String, String> parameters = Collections.emptyMap(); > + Assert.assertEquals("?width=200&height=170&keepAspectRatio=true", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when both the width and the height are unspecified and only the > + * image width is limited in the configuration. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothWidthAndHeightAreUnspecifiedAndOnlyImageWidthIsLimited() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + oneOf(configuration).getImageWidthLimit(); > + will(returnValue(25)); > + oneOf(configuration).getImageHeightLimit(); > + will(returnValue(-1)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("width", "45%"); > + parameters.put("style", "height:10em"); > + Assert.assertEquals("?width=25", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when both the width and the height are unspecified and the image > + * size is not limited in the configuration. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothWidthAndHeightAreUnspecifiedAndImageSizeIsNotLimited() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + oneOf(configuration).getImageWidthLimit(); > + will(returnValue(-1)); > + oneOf(configuration).getImageHeightLimit(); > + will(returnValue(-1)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("style", "bad CSS declaration"); > + Assert.assertEquals("", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when the attachment URL has a fragment identifier. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenAttachmentURLHasFragmentIdentifier() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("test#fragment")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("width", "23"); > + Assert.assertEquals("test?width=23#fragment", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when the attachment URL has a query string and a fragment > + * identifier. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenAttachmentURLHasQueryStringAndFragmentIdentifier() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("test?param=value#fragment")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("height", "17"); > + Assert.assertEquals("test?param=value&height=17#fragment", wikiModel.getImageURL("", "", parameters)); > + } > + > + /** > + * Tests that the proper image URL is generated when both the style and the dimension parameters are specified. > + * > + * @throws Exception if an exception occurs while running the test > + */ > + @Test > + public void testGetImageURLWhenBothStyleAndDimensionParametersAreSpecified() throws Exception > + { > + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); > + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); > + getMockery().checking(new Expectations() > + { > + { > + oneOf(documentAccessBridge).getAttachmentURL("", ""); > + will(returnValue("")); > + oneOf(configuration).isIncludeImageDimensionsInImageURL(); > + will(returnValue(true)); > + } > + }); > + Map<String, String> parameters = new HashMap<String, String>(); > + parameters.put("height", "46"); > + parameters.put("width", "101px"); > + parameters.put("style", "width: 20%; height:75px"); > + // Note that the style parameter take precedence over the dimension parameters and the width is actually 20% but > + // we can't use it for resizing the image on the server side so it's omitted from the query string. > + Assert.assertEquals("?height=75", wikiModel.getImageURL("", "", parameters)); > + } > }

15 years

2
1
0 0

Re: [xwiki-devs] [xwiki-notifications] r31022 - platform/web/trunk/standard/src/main/webapp/templates

by Sergiu Dumitriu

On 09/09/2010 01:59 PM, cjdelisle (SVN) wrote: > Author: cjdelisle > Date: 2010-09-09 13:59:55 +0200 (Thu, 09 Sep 2010) > New Revision: 31022 > > Modified: > platform/web/trunk/standard/src/main/webapp/templates/commentsinline.vm > Log: > XWIKI-2110: Made comments be rendered in the context of the document "XWiki.XWikiComments" > > Modified: platform/web/trunk/standard/src/main/webapp/templates/commentsinline.vm > =================================================================== > --- platform/web/trunk/standard/src/main/webapp/templates/commentsinline.vm 2010-09-09 08:37:05 UTC (rev 31021) > +++ platform/web/trunk/standard/src/main/webapp/templates/commentsinline.vm 2010-09-09 11:59:55 UTC (rev 31022) > @@ -110,7 +110,7 @@ > ## > ## > ## > -#macro(displayThread $key $commentThreads) > +#macro(displayThread $key $commentThreads, $commentDoc) > ## If request parameter replyto is specified then we show the form under that comment. > #if($replyTo == "$key") > #displayCommentForm() > @@ -134,7 +134,7 @@ > ## > ## > ## > -#macro(displayComment $comment) > +#macro(displayComment $comment, $commentDoc) > <div id="xwikicomment_${comment.number}" class="xwikicomment#if($comment.getProperty('author').value == $doc.creator) commentByCreator#end"> > ##<div class="commentavatar">#useravatar($comment.author)</div> > <div class="commentheader"> > @@ -156,7 +156,7 @@ > #end > </span>## commenttools > </div>## commentheader > -<div class="commentcontent">$doc.display('comment', 'view', $comment)</div> > +<div class="commentcontent">$commentDoc.getRenderedContent($comment.getProperty('comment').getValue(), $doc.getSyntax().toIdString())</div> > </div>## xwikicomment > #end > ## > @@ -302,7 +302,11 @@ > #foreach($comment in $comments) > #addCommentToThreads($comment, $commentThreads, $rootKey) > #end > - #displayThread($rootKey, $commentThreads) > + ## > + ## Comments will be rendered in the context of this document. > + #set($commentDoc = $xwiki.getDocument('XWiki.XWikiComments')) What if the user does not have the right to view XWiki.XWikiComments? $commentDoc will be null, and the display will fail. You should fall back to $doc in this case. Note that this is not so absurd, since it's a good practice to forbid access to the XWiki space. One incompatible use case is that all code using $doc will show different results now, but I don't think this is a very valid use case. > + ## > + #displayThread($rootKey, $commentThreads, $commentDoc) > #else > <p class="noitems">$msg.get('core.viewers.comments.noComments')</p> > #end -- Sergiu Dumitriu http://purl.org/net/sergiu/

15 years

3
2
0 0

Re: [xwiki-devs] [xwiki-notifications] r31053 - in platform/core/trunk/xwiki-rendering: xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration xwiki-rendering-a

by Thomas Mortagne

On Mon, Sep 13, 2010 at 09:08, Vincent Massol <vincent(a)massol.net> wrote: > Hi Marius, > > I think there are some changes we need to discuss in this commit: > - new config api methods > - new wiki model api methods > - new deps introduced in the rendering module (makes it harder to use without maven) > > These 2 are important as they are important public apis. The WikiModel one is the most important and we need to be 100% sure we want to add a new methods. There must absolutely be the minimal number of methods in this class. Also we're going to break people using the rendering module outside of xwiki (guys from eXo for example, so we need to be careful). > > I haven't thought too much about it yet but my first feeling is that the methods added look quite specific. I'd like to have Thomas's opinion on this since he's the one who's the most knowledgeable about the rendering module. > > Thanks > -Vincent > > On Sep 12, 2010, at 1:30 PM, mflorea (SVN) wrote: > >> Author: mflorea >> Date: 2010-09-12 13:30:15 +0200 (Sun, 12 Sep 2010) >> New Revision: 31053 >> >> Modified: >> platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java >> platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java >> platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java >> platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java >> platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml >> platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java >> platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java >> platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java >> platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java >> Log: >> XWIKI-5479: Include image dimensions in the image URL for server side image resizing >> >> Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java >> =================================================================== >> --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java 2010-09-11 17:15:46 UTC (rev 31052) >> +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/configuration/RenderingConfiguration.java 2010-09-12 11:30:15 UTC (rev 31053) >> @@ -50,7 +50,7 @@ >> * </ul> >> * Note that if the page title is empty or not defined then it defaults to %p. This is also the case >> * if the title cannot be retrieved for the document. >> - * >> + * >> * The default is "%p". Some examples: "%s.%p", "%w:%s.%p". >> * >> * @return the format to use to display link labels when the user hasn't specified a label >> @@ -63,4 +63,42 @@ >> * descriptor (ie defined by the macro author) will be used >> */ >> Properties getMacroCategories(); >> + >> + /** >> + * @return {@code true} to include the image dimensions extracted from the image parameters in the image URL, >> + * {@code false} otherwise; when image dimensions are included in the image URL the image can be resized >> + * on the server side before being downloaded. >> + * @since 2.5M2 >> + */ >> + boolean isIncludeImageDimensionsInImageURL(); >> + >> + /** >> + * One way to improve page load speed is to resize images on the server side just before rendering the page. The >> + * rendering module can use the image width provided by the user to scale the image. When the user doesn't specify >> + * the image width the rendering module can limit the width of the image based on this configuration parameter. >> + * <p> >> + * The default value is {@code -1} which means image width is not limited by default. Use a value greater than 0 to >> + * limit the image width (pixels). Note that the aspect ratio is kept even when both the width and the height of the >> + * image are limited. >> + * >> + * @return the maximum image width when there's no user supplied width >> + * @see #isIncludeImageDimensionsInImageURL() >> + * @since 2.5M2 >> + */ >> + int getImageWidthLimit(); >> + >> + /** >> + * One way to improve page load speed is to resize images on the server side just before rendering the page. The >> + * rendering module can use the image height provided by the user to scale the image. When the user doesn't specify >> + * the image height the rendering module can limit the height of the image based on this configuration parameter. >> + * <p> >> + * The default value is {@code -1} which means image height is not limited by default. Use a value greater than 0 to >> + * limit the image height (pixels). Note that the aspect ratio is kept even when both the width and the height of >> + * the image are limited. >> + * >> + * @return the maximum image height when there's no user supplied height >> + * @see #isIncludeImageDimensionsInImageURL() >> + * @since 2.5M2 >> + */ >> + int getImageHeightLimit(); >> } This is very wrong. Theses configuration is supposed to be a rendering-xwiki specific thing so it has nothing to do in the standard rendering configuration. >> >> Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java >> =================================================================== >> --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java 2010-09-11 17:15:46 UTC (rev 31052) >> +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/configuration/DefaultRenderingConfiguration.java 2010-09-12 11:30:15 UTC (rev 31053) >> @@ -45,6 +45,21 @@ >> private Properties macroCategories = new Properties(); >> >> /** >> + * @see #getImageWidthLimit() >> + */ >> + private int imageWidthLimit = -1; >> + >> + /** >> + * @see #getImageHeightLimit() >> + */ >> + private int imageHeightLimit = -1; >> + >> + /** >> + * @see #isIncludeImageDimensionsInImageURL() >> + */ >> + private boolean includeImageDimensionsInImageURL = true; >> + >> + /** >> * {@inheritDoc} >> * >> * @see org.xwiki.rendering.configuration.RenderingConfiguration#getLinkLabelFormat() >> @@ -84,4 +99,66 @@ >> // to work even without a configuration store. >> this.macroCategories.setProperty(macroId.toString(), category); >> } >> + >> + /** >> + * {@inheritDoc} >> + * >> + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageWidthLimit() >> + */ >> + public int getImageWidthLimit() >> + { >> + return this.imageWidthLimit; >> + } >> + >> + /** >> + * @param imageWidthLimit the maximum image width when there's no user supplied width >> + */ >> + public void setImageWidthLimit(int imageWidthLimit) >> + { >> + // This method is useful for those using the XWiki Rendering in standalone mode since it allows the rendering >> + // to work even without a configuration store. >> + this.imageWidthLimit = imageWidthLimit; >> + } >> + >> + /** >> + * {@inheritDoc} >> + * >> + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageHeightLimit() >> + */ >> + public int getImageHeightLimit() >> + { >> + return this.imageHeightLimit; >> + } >> + >> + /** >> + * @param imageHeightLimit the maximum image height when there's no user supplied height >> + */ >> + public void setImageHeightLimit(int imageHeightLimit) >> + { >> + // This method is useful for those using the XWiki Rendering in standalone mode since it allows the rendering >> + // to work even without a configuration store. >> + this.imageHeightLimit = imageHeightLimit; >> + } >> + >> + /** >> + * {@inheritDoc} >> + * >> + * @see org.xwiki.rendering.configuration.RenderingConfiguration#isIncludeImageDimensionsInImageURL() >> + */ >> + public boolean isIncludeImageDimensionsInImageURL() >> + { >> + return this.includeImageDimensionsInImageURL; >> + } >> + >> + /** >> + * @param includeImageDimensionsInImageURL {@code true} to include image dimensions extracted from the image >> + * parameters in the image URL so that the image can be resized on the server side before being >> + * downloaded, {@code false} otherwise >> + */ >> + public void setIncludeImageDimensionsInImageURL(boolean includeImageDimensionsInImageURL) >> + { >> + // This method is useful for those using the XWiki Rendering in standalone mode since it allows the rendering >> + // to work even without a configuration store. >> + this.includeImageDimensionsInImageURL = includeImageDimensionsInImageURL; >> + } >> } >> >> Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java >> =================================================================== >> --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java 2010-09-11 17:15:46 UTC (rev 31052) >> +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/internal/renderer/xhtml/DefaultXHTMLImageRenderer.java 2010-09-12 11:30:15 UTC (rev 31053) >> @@ -42,10 +42,9 @@ >> * Default implementation for rendering images as XHTML. We handle both cases: >> * <ul> >> * <li>when inside a wiki (ie when an implementation of {@link WikiModel} is provided.</li> >> - * <li>when outside of a wiki. In this case we only handle external images and document images don't >> - * display anything.</li> >> + * <li>when outside of a wiki. In this case we only handle external images and document images don't display anything.</li> >> * </ul> >> - * >> + * >> * @version $Id$ >> * @since 2.0M3 >> */ >> @@ -68,6 +67,7 @@ >> >> /** >> * {@inheritDoc} >> + * >> * @see Initializable#initialize() >> */ >> public void initialize() throws InitializationException >> @@ -83,7 +83,7 @@ >> >> /** >> * {@inheritDoc} >> - * >> + * >> * @see XHTMLImageRenderer#setXHTMLWikiPrinter(XHTMLWikiPrinter) >> */ >> public void setXHTMLWikiPrinter(XHTMLWikiPrinter printer) >> @@ -103,7 +103,7 @@ >> >> /** >> * {@inheritDoc} >> - * >> + * >> * @see XHTMLImageRenderer#onImage(Image, boolean, Map) >> */ >> public void onImage(Image image, boolean isFreeStandingURI, Map<String, String> parameters) >> @@ -118,8 +118,9 @@ >> // that would not honor this contract. >> if (this.wikiModel != null) { >> DocumentImage documentImage = DocumentImage.class.cast(image); >> - imageURL = this.wikiModel.getAttachmentURL(documentImage.getDocumentName(), >> - documentImage.getAttachmentName()); >> + imageURL = >> + this.wikiModel.getImageURL(documentImage.getDocumentName(), documentImage.getAttachmentName(), >> + parameters); >> } else { >> throw new RuntimeException("Invalid Image type. In non wiki mode, all image types must be URL images."); >> } >> >> Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java >> =================================================================== >> --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java 2010-09-11 17:15:46 UTC (rev 31052) >> +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/wiki/WikiModel.java 2010-09-12 11:30:15 UTC (rev 31053) >> @@ -19,13 +19,15 @@ >> */ >> package org.xwiki.rendering.wiki; >> >> +import java.util.Map; >> + >> import org.xwiki.component.annotation.ComponentRole; >> >> /** >> * Bridge between the Rendering module and a Wiki Model. Contains wiki APIs required by Rendering classes such as >> - * Renderers. For example the XHTML Link Renderer needs to know if a wiki document exists in order to know how to >> - * generate the HTML (in order to display a question mark for non existing documents) and it also needs to get the >> - * URL pointing the wiki document. >> + * Renderers. For example the XHTML Link Renderer needs to know if a wiki document exists in order to know how to >> + * generate the HTML (in order to display a question mark for non existing documents) and it also needs to get the URL >> + * pointing the wiki document. >> * >> * @version $Id$ >> * @since 2.0M1 >> @@ -34,36 +36,48 @@ >> public interface WikiModel >> { >> /** >> - * @param documentName the document name of the document containing the attachment. The syntax used depends on >> - * the underlying wiki system used. For example for XWiki a valid documentName would be >> - * {@code wiki:Space.Page} >> + * @param documentName the document name of the document containing the attachment. The syntax used depends on the >> + * underlying wiki system used. For example for XWiki a valid documentName would be >> + * {@code wiki:Space.Page} >> * @param attachmentName the name of the attachment in the passed document wikip page >> * @return the URL to the attachment >> */ >> String getAttachmentURL(String documentName, String attachmentName); >> >> /** >> - * @param documentName the document name as a String. The syntax used depends on >> - * the underlying wiki system used. For example for XWiki a valid documentName would be >> - * {@code wiki:Space.Page} >> + * Generate image specific URL. The difference with {@link #getAttachmentURL(String, String)} is that in some >> + * implementation we want to make a distinction between displayed image and a simple link targeting an attachment >> + * file. >> + * >> + * @param documentName the document name of the document containing the attachment. The syntax used depends on the >> + * underlying wiki system used. For example for XWiki a valid documentName would be >> + * {@code wiki:Space.Page} >> + * @param attachmentName the name of the attachment in the passed document wiki page >> + * @param parameters custom parameters >> + * @return the URL to the image >> + * @since 2.5M2 >> + */ >> + String getImageURL(String documentName, String attachmentName, Map<String, String> parameters); >> + >> + /** >> + * @param documentName the document name as a String. The syntax used depends on the underlying wiki system used. >> + * For example for XWiki a valid documentName would be {@code wiki:Space.Page} >> * @return true if the document exists and can be viewed or false otherwise >> */ >> boolean isDocumentAvailable(String documentName); >> >> /** >> - * @param documentName the document name as a String. The syntax used depends on >> - * the underlying wiki system used. For example for XWiki a valid documentName would be >> - * {@code wiki:Space.Page} >> + * @param documentName the document name as a String. The syntax used depends on the underlying wiki system used. >> + * For example for XWiki a valid documentName would be {@code wiki:Space.Page} >> * @param anchor an anchor pointing to some place inside the document or null >> * @param queryString a query string specifying some parameters or null >> * @return the URL to view the specified wiki document >> */ >> String getDocumentViewURL(String documentName, String anchor, String queryString); >> - >> + >> /** >> - * @param documentName the document name as a String. The syntax used depends on >> - * the underlying wiki system used. For example for XWiki a valid documentName would be >> - * {@code wiki:Space.Page} >> + * @param documentName the document name as a String. The syntax used depends on the underlying wiki system used. >> + * For example for XWiki a valid documentName would be {@code wiki:Space.Page} >> * @param anchor an anchor pointing to some place inside the document or null >> * @param queryString a query string specifying some parameters or null >> * @return the URL to edit the specified wiki document >> >> Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml >> =================================================================== >> --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml 2010-09-11 17:15:46 UTC (rev 31052) >> +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/pom.xml 2010-09-12 11:30:15 UTC (rev 31053) >> @@ -54,6 +54,12 @@ >> <artifactId>xwiki-core-script</artifactId> >> <version>${pom.version}</version> >> </dependency> >> + <dependency> >> +  >> + <groupId>net.sourceforge.cssparser</groupId> >> + <artifactId>cssparser</artifactId> >> + <version>0.9.5</version> >> + </dependency> >>  >> <dependency> >> <groupId>org.xwiki.platform</groupId> >> >> Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java >> =================================================================== >> --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java 2010-09-11 17:15:46 UTC (rev 31052) >> +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfiguration.java 2010-09-12 11:30:15 UTC (rev 31053) >> @@ -19,13 +19,13 @@ >> */ >> package org.xwiki.rendering.internal.configuration; >> >> +import java.util.Properties; >> + >> import org.xwiki.component.annotation.Component; >> import org.xwiki.component.annotation.Requirement; >> import org.xwiki.configuration.ConfigurationSource; >> import org.xwiki.rendering.configuration.RenderingConfiguration; >> >> -import java.util.Properties; >> - >> /** >> * All configuration options for the rendering subsystem. >> * >> @@ -46,7 +46,7 @@ >> private static final String DEFAULT_LINK_LABEL_FORMAT = "%p"; >> >> /** >> - * Defines from where to read the rendering configuration data. >> + * Defines from where to read the rendering configuration data. >> */ >> @Requirement >> private ConfigurationSource configuration; >> @@ -58,16 +58,46 @@ >> */ >> public String getLinkLabelFormat() >> { >> - return this.configuration.getProperty(PREFIX + "linkLabelFormat", DEFAULT_LINK_LABEL_FORMAT); >> + return this.configuration.getProperty(PREFIX + "linkLabelFormat", DEFAULT_LINK_LABEL_FORMAT); >> } >> >> /** >> * {@inheritDoc} >> - * >> + * >> * @see org.xwiki.rendering.configuration.RenderingConfiguration#getMacroCategories() >> */ >> public Properties getMacroCategories() >> { >> return this.configuration.getProperty(PREFIX + "macroCategories", Properties.class); >> } >> + >> + /** >> + * {@inheritDoc} >> + * >> + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageWidthLimit() >> + */ >> + public int getImageWidthLimit() >> + { >> + return this.configuration.getProperty(PREFIX + "imageWidthLimit", -1); >> + } >> + >> + /** >> + * {@inheritDoc} >> + * >> + * @see org.xwiki.rendering.configuration.RenderingConfiguration#getImageHeightLimit() >> + */ >> + public int getImageHeightLimit() >> + { >> + return this.configuration.getProperty(PREFIX + "imageHeightLimit", -1); >> + } >> + >> + /** >> + * {@inheritDoc} >> + * >> + * @see org.xwiki.rendering.configuration.RenderingConfiguration#isIncludeImageDimensionsInImageURL() >> + */ >> + public boolean isIncludeImageDimensionsInImageURL() >> + { >> + return this.configuration.getProperty(PREFIX + "includeImageDimensionsInImageURL", true); >> + } >> } >> >> Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java >> =================================================================== >> --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java 2010-09-11 17:15:46 UTC (rev 31052) >> +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/main/java/org/xwiki/rendering/internal/wiki/XWikiWikiModel.java 2010-09-12 11:30:15 UTC (rev 31053) >> @@ -19,17 +19,25 @@ >> */ >> package org.xwiki.rendering.internal.wiki; >> >> +import java.io.IOException; >> +import java.io.StringReader; >> import java.io.UnsupportedEncodingException; >> import java.net.URLEncoder; >> +import java.util.Map; >> >> import org.apache.commons.lang.StringUtils; >> +import org.w3c.css.sac.InputSource; >> +import org.w3c.dom.css.CSSStyleDeclaration; >> import org.xwiki.bridge.DocumentAccessBridge; >> -import org.xwiki.model.reference.DocumentReference; >> import org.xwiki.component.annotation.Component; >> import org.xwiki.component.annotation.Requirement; >> +import org.xwiki.model.reference.DocumentReference; >> import org.xwiki.model.reference.EntityReferenceSerializer; >> +import org.xwiki.rendering.configuration.RenderingConfiguration; >> import org.xwiki.rendering.wiki.WikiModel; >> >> +import com.steadystate.css.parser.CSSOMParser; >> + >> /** >> * Implementation using the Document Access Bridge ({@link DocumentAccessBridge}). >> * >> @@ -39,14 +47,47 @@ >> @Component >> public class XWikiWikiModel implements WikiModel >> { >> + /** >> + * The suffix used to mark an amount of pixels. >> + */ >> + private static final String PIXELS = "px"; >> + >> + /** >> + * The name of the {@code width} image parameter. >> + */ >> + private static final String WIDTH = "width"; >> + >> + /** >> + * The name of the {@code height} image parameter. >> + */ >> + private static final String HEIGHT = "height"; >> + >> + /** >> + * The component used to access configuration parameters. >> + */ >> @Requirement >> + private RenderingConfiguration renderingConfiguration; >> + >> + /** >> + * The component used to access the underlying XWiki model. >> + */ >> + @Requirement >> private DocumentAccessBridge documentAccessBridge; >> >> + /** >> + * The component used to serialize entity references to strings. >> + */ >> @Requirement >> private EntityReferenceSerializer<String> entityReferenceSerializer; >> >> /** >> + * The object used to parse the CSS from the image style parameter. >> + */ >> + private final CSSOMParser cssParser = new CSSOMParser(); >> + >> + /** >> * {@inheritDoc} >> + * >> * @see WikiModel#getAttachmentURL(String, String) >> */ >> public String getAttachmentURL(String documentName, String attachmentName) >> @@ -55,7 +96,112 @@ >> } >> >> /** >> + * Extracts the specified image dimension from the image parameters. >> + * >> + * @param dimension either {@code width} or {@code height} >> + * @param imageParameters the image parameters; may include the {@code width}, {@code height} and {@code style} >> + * parameters >> + * @return the value of the passed dimension if it is specified in the image parameters, {@code null} otherwise >> + */ >> + private String getImageDimension(String dimension, Map<String, String> imageParameters) >> + { >> + // Check first if the style parameter contains information about the given dimension. In-line style has priority >> + // over the dimension parameters. >> + String value = null; >> + String style = imageParameters.get("style"); >> + if (StringUtils.isNotBlank(style)) { >> + try { >> + CSSStyleDeclaration sd = cssParser.parseStyleDeclaration(new InputSource(new StringReader(style))); >> + value = sd.getPropertyValue(dimension); >> + } catch (IOException e) { >> + // Ignore the style parameter. >> + } >> + } >> + if (StringUtils.isBlank(value)) { >> + // Fall back on the value of the dimension parameter. >> + value = imageParameters.get(dimension); >> + } >> + return value; >> + } >> + >> + /** >> + * Creates the query string that can be added to an image URL to resize the image on the server side. >> + * >> + * @param imageParameters image parameters, including width and height then they are specified >> + * @return the query string to be added to an image URL in order to resize the image on the server side >> + */ >> + private StringBuilder getImageURLQueryString(Map<String, String> imageParameters) >> + { >> + String width = StringUtils.chomp(getImageDimension(WIDTH, imageParameters), PIXELS); >> + String height = StringUtils.chomp(getImageDimension(HEIGHT, imageParameters), PIXELS); >> + boolean useHeight = StringUtils.isNotEmpty(height) && StringUtils.isNumeric(height); >> + StringBuilder queryString = new StringBuilder(); >> + if (StringUtils.isEmpty(width) || !StringUtils.isNumeric(width)) { >> + // Width is unspecified or is not measured in pixels. >> + if (useHeight) { >> + // Height is specified in pixels. >> + queryString.append('&').append(HEIGHT).append('=').append(height); >> + } else { >> + // If image width and height are unspecified or if they are not expressed in pixels then limit the image >> + // size to best fit the rectangle specified in the configuration (keeping aspect ratio). >> + int widthLimit = renderingConfiguration.getImageWidthLimit(); >> + if (widthLimit > 0) { >> + queryString.append('&').append(WIDTH).append('=').append(widthLimit); >> + } >> + int heightLimit = renderingConfiguration.getImageHeightLimit(); >> + if (heightLimit > 0) { >> + queryString.append('&').append(HEIGHT).append('=').append(heightLimit); >> + } >> + if (widthLimit > 0 && heightLimit > 0) { >> + queryString.append("&keepAspectRatio=").append(true); >> + } >> + } >> + } else { >> + // Width is specified in pixels. >> + queryString.append('&').append(WIDTH).append('=').append(width); >> + if (useHeight) { >> + // Height is specified in pixels. >> + queryString.append('&').append(HEIGHT).append('=').append(height); >> + } >> + } >> + return queryString; >> + } >> + >> + /** >> * {@inheritDoc} >> + * >> + * @see org.xwiki.rendering.wiki.WikiModel#getImageURL(java.lang.String, java.lang.String, java.util.Map) >> + */ >> + public String getImageURL(String documentName, String attachmentName, Map<String, String> parameters) >> + { >> + String url = getAttachmentURL(documentName, attachmentName); >> + if (!renderingConfiguration.isIncludeImageDimensionsInImageURL()) { >> + return url; >> + } >> + >> + StringBuilder queryString = getImageURLQueryString(parameters); >> + if (queryString.length() == 0) { >> + return url; >> + } >> + >> + // Determine the insertion point. >> + int insertionPoint = url.lastIndexOf('#'); >> + if (insertionPoint < 0) { >> + // No fragment identifier. >> + insertionPoint = url.length(); >> + } >> + if (url.lastIndexOf('?', insertionPoint) < 0) { >> + // No query string. >> + queryString.setCharAt(0, '?'); >> + } >> + >> + // Insert the query string. >> + return new StringBuilder(url).insert(insertionPoint, queryString).toString(); >> + } >> + >> + /** >> + * {@inheritDoc} >> + * >> * @see WikiModel#isDocumentAvailable(String) >> */ >> public boolean isDocumentAvailable(String documentName) >> @@ -65,6 +211,7 @@ >> >> /** >> * {@inheritDoc} >> + * >> * @see WikiModel#getDocumentViewURL(String, String, String) >> */ >> public String getDocumentViewURL(String documentName, String anchor, String queryString) >> @@ -74,6 +221,7 @@ >> >> /** >> * {@inheritDoc} >> + * >> * @see WikiModel#getDocumentEditURL(String, String, String) >> */ >> public String getDocumentEditURL(String documentName, String anchor, String queryString) >> @@ -89,17 +237,17 @@ >> // See http://www.w3.org/TR/html40/appendix/notes.html#non-ascii-chars >> // TODO: Once the xwiki-url module is usable, refactor this code to use it and remove the need to >> // perform explicit encoding here. >> - modifiedQueryString = >> + modifiedQueryString = >> "parent=" + URLEncoder.encode(this.entityReferenceSerializer.serialize(reference), "UTF-8"); >> } catch (UnsupportedEncodingException e) { >> // Not supporting UTF-8 as a valid encoding for some reasons. We consider XWiki cannot work >> // without that encoding. >> - throw new RuntimeException("Failed to URL encode [" + this.entityReferenceSerializer.serialize( >> - reference) + "] using UTF-8.", e); >> + throw new RuntimeException("Failed to URL encode [" >> + + this.entityReferenceSerializer.serialize(reference) + "] using UTF-8.", e); >> } >> } >> } >> - >> + >> return this.documentAccessBridge.getURL(documentName, "create", modifiedQueryString, anchor); >> } >> } >> >> Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java >> =================================================================== >> --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java 2010-09-11 17:15:46 UTC (rev 31052) >> +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/configuration/XWikiRenderingConfigurationTest.java 2010-09-12 11:30:15 UTC (rev 31053) >> @@ -22,9 +22,9 @@ >> import java.util.Properties; >> >> import org.jmock.Expectations; >> -import org.xwiki.configuration.ConfigurationSource; >> import org.junit.Assert; >> import org.junit.Test; >> +import org.xwiki.configuration.ConfigurationSource; >> import org.xwiki.test.AbstractMockingComponentTestCase; >> import org.xwiki.test.annotation.MockingRequirement; >> >> @@ -43,10 +43,13 @@ >> public void testGetLinkLabelFormat() throws Exception >> { >> final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); >> - getMockery().checking(new Expectations() {{ >> - allowing(source).getProperty("rendering.linkLabelFormat", "%p"); >> + getMockery().checking(new Expectations() >> + { >> + { >> + allowing(source).getProperty("rendering.linkLabelFormat", "%p"); >> will(returnValue("%p")); >> - }}); >> + } >> + }); >> >> Assert.assertEquals("%p", this.configuration.getLinkLabelFormat()); >> } >> @@ -55,12 +58,60 @@ >> public void testGetMacroCategories() throws Exception >> { >> final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); >> - getMockery().checking(new Expectations() {{ >> - allowing(source).getProperty("rendering.macroCategories", Properties.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + allowing(source).getProperty("rendering.macroCategories", Properties.class); >> will(returnValue(new Properties())); >> - }}); >> + } >> + }); >> >> Assert.assertNotNull(this.configuration.getMacroCategories()); >> Assert.assertEquals(0, this.configuration.getMacroCategories().size()); >> } >> + >> + @Test >> + public void testGetImageWidthLimit() throws Exception >> + { >> + final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + allowing(source).getProperty("rendering.imageWidthLimit", -1); >> + will(returnValue(100)); >> + } >> + }); >> + >> + Assert.assertEquals(100, this.configuration.getImageWidthLimit()); >> + } >> + >> + @Test >> + public void testGetImageHeightLimit() throws Exception >> + { >> + final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + allowing(source).getProperty("rendering.imageHeightLimit", -1); >> + will(returnValue(150)); >> + } >> + }); >> + >> + Assert.assertEquals(150, this.configuration.getImageHeightLimit()); >> + } >> + >> + @Test >> + public void testIsIncludeImageDimensionsInImageURL() throws Exception >> + { >> + final ConfigurationSource source = getComponentManager().lookup(ConfigurationSource.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + allowing(source).getProperty("rendering.includeImageDimensionsInImageURL", true); >> + will(returnValue(false)); >> + } >> + }); >> + >> + Assert.assertFalse(this.configuration.isIncludeImageDimensionsInImageURL()); >> + } >> } >> >> Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java >> =================================================================== >> --- platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java 2010-09-11 17:15:46 UTC (rev 31052) >> +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-xwiki/src/test/java/org/xwiki/rendering/internal/wiki/XWikiWikiModelTest.java 2010-09-12 11:30:15 UTC (rev 31053) >> @@ -19,12 +19,20 @@ >> */ >> package org.xwiki.rendering.internal.wiki; >> >> +import java.util.Collections; >> +import java.util.HashMap; >> +import java.util.Map; >> + >> +import junit.framework.Assert; >> + >> import org.jmock.Expectations; >> -import org.jmock.Mockery; >> +import org.junit.Test; >> import org.xwiki.bridge.DocumentAccessBridge; >> import org.xwiki.model.reference.DocumentReference; >> -import org.xwiki.component.util.ReflectionUtils; >> import org.xwiki.model.reference.EntityReferenceSerializer; >> +import org.xwiki.rendering.configuration.RenderingConfiguration; >> +import org.xwiki.test.AbstractMockingComponentTestCase; >> +import org.xwiki.test.annotation.MockingRequirement; >> >> /** >> * Unit tests for {@link XWikiWikiModel}. >> @@ -32,31 +40,321 @@ >> * @version $Id$ >> * @since 2.0M1 >> */ >> -public class XWikiWikiModelTest >> +public class XWikiWikiModelTest extends AbstractMockingComponentTestCase >> { >> - @org.junit.Test >> + @MockingRequirement >> + private XWikiWikiModel wikiModel; >> + >> + @Test >> public void testGetDocumentEditURLWhenNoQueryStringSpecified() throws Exception >> { >> - Mockery mockery = new Mockery(); >> - XWikiWikiModel wikiModel = new XWikiWikiModel(); >> + final EntityReferenceSerializer< ? > entityReferenceSerializer = >> + getComponentManager().lookup(EntityReferenceSerializer.class); >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final DocumentReference documentReference = new DocumentReference("wiki", "Space", "Page"); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getCurrentDocumentReference(); >> + will(returnValue(documentReference)); >> + oneOf(entityReferenceSerializer).serialize(documentReference); >> + will(returnValue("wiki:Space.Page\u20AC")); >> >> - final EntityReferenceSerializer mockEntityReferenceSerializer = mockery.mock(EntityReferenceSerializer.class); >> - ReflectionUtils.setFieldValue(wikiModel, "entityReferenceSerializer", mockEntityReferenceSerializer); >> + // The test is here: we verify that getURL is called with the query string already encoded since >> + // getURL() doesn't encode it. >> + oneOf(documentAccessBridge).getURL("Space.Page\u20AC", "create", "parent=wiki%3ASpace.Page%E2%82%AC", >> + "anchor"); >> + } >> + }); >> >> - final DocumentAccessBridge mockDocumentAccessBridge = mockery.mock(DocumentAccessBridge.class); >> - ReflectionUtils.setFieldValue(wikiModel, "documentAccessBridge", mockDocumentAccessBridge); >> + wikiModel.getDocumentEditURL("Space.Page\u20AC", "anchor", null); >> + } >> >> - final DocumentReference docReference = new DocumentReference("wiki", "Space", "Page"); >> - mockery.checking(new Expectations() {{ >> - oneOf(mockDocumentAccessBridge).getCurrentDocumentReference(); will(returnValue(docReference)); >> - oneOf(mockEntityReferenceSerializer).serialize(docReference); will(returnValue("wiki:Space.Page\u20AC")); >> + /** >> + * Tests that the proper image URL is generated when both the width and the height image parameters are specified. >> + * >> + * @throws Exception if an exception occurs while running the test >> + */ >> + @Test >> + public void testGetImageURLWhenBothWidthAndHeightAttributesAreSpecified() throws Exception >> + { >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getAttachmentURL("", ""); >> + will(returnValue("")); >> + oneOf(configuration).isIncludeImageDimensionsInImageURL(); >> + will(returnValue(true)); >> + } >> + }); >> + Map<String, String> parameters = new HashMap<String, String>(); >> + parameters.put("width", "100px"); >> + parameters.put("height", "50"); >> + Assert.assertEquals("?width=100&height=50", wikiModel.getImageURL("", "", parameters)); >> + } >> >> - // The test is here: we verify that getURL is called with the query string already encoded since getURL() >> - // doesn't encode it. >> - oneOf(mockDocumentAccessBridge).getURL("Space.Page\u20AC", "create", "parent=wiki%3ASpace.Page%E2%82%AC", >> - "anchor"); >> - }}); >> + /** >> + * Tests that the proper image URL is generated when both the width and the height image parameters are specified >> + * but including them in the image URL is forbidden from the rendering configuration. >> + * >> + * @throws Exception if an exception occurs while running the test >> + */ >> + @Test >> + public void testGetImageURLWhenIncludingImageDimensionsIsForbidden() throws Exception >> + { >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getAttachmentURL("", ""); >> + will(returnValue("")); >> + oneOf(configuration).isIncludeImageDimensionsInImageURL(); >> + will(returnValue(false)); >> + } >> + }); >> + Map<String, String> parameters = new HashMap<String, String>(); >> + parameters.put("width", "101px"); >> + parameters.put("height", "55px"); >> + Assert.assertEquals("", wikiModel.getImageURL("", "", parameters)); >> + } >> >> - wikiModel.getDocumentEditURL("Space.Page\u20AC", "anchor", null); >> + /** >> + * Tests that the proper image URL is generated when both the width and the height CSS properties are specified. >> + * >> + * @throws Exception if an exception occurs while running the test >> + */ >> + @Test >> + public void testGetImageURLWhenBothWidthAndHeightCSSPropertiesAreSpecified() throws Exception >> + { >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getAttachmentURL("", ""); >> + will(returnValue("")); >> + oneOf(configuration).isIncludeImageDimensionsInImageURL(); >> + will(returnValue(true)); >> + } >> + }); >> + Map<String, String> parameters = new HashMap<String, String>(); >> + parameters.put("style", "border: 1px; height: 30px; margin-top: 2em; width: 70px"); >> + Assert.assertEquals("?width=70&height=30", wikiModel.getImageURL("", "", parameters)); >> } >> + >> + /** >> + * Tests that the proper image URL is generated when only the width image parameter is specified. >> + * >> + * @throws Exception if an exception occurs while running the test >> + */ >> + @Test >> + public void testGetImageURLWhenOnlyWidthAttributeIsSpecified() throws Exception >> + { >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getAttachmentURL("", ""); >> + will(returnValue("")); >> + oneOf(configuration).isIncludeImageDimensionsInImageURL(); >> + will(returnValue(true)); >> + } >> + }); >> + Map<String, String> parameters = new HashMap<String, String>(); >> + parameters.put("width", "150"); >> + parameters.put("height", "30%"); >> + Assert.assertEquals("?width=150", wikiModel.getImageURL("", "", parameters)); >> + } >> + >> + /** >> + * Tests that the proper image URL is generated when only the height CSS property is specified. >> + * >> + * @throws Exception if an exception occurs while running the test >> + */ >> + @Test >> + public void testGetImageURLWhenOnlyHeightCSSPropertyIsSpecified() throws Exception >> + { >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getAttachmentURL("", ""); >> + will(returnValue("")); >> + oneOf(configuration).isIncludeImageDimensionsInImageURL(); >> + will(returnValue(true)); >> + } >> + }); >> + Map<String, String> parameters = new HashMap<String, String>(); >> + parameters.put("style", "width: 5cm; height: 80px"); >> + Assert.assertEquals("?height=80", wikiModel.getImageURL("", "", parameters)); >> + } >> + >> + /** >> + * Tests that the proper image URL is generated when both the width and the height are unspecified and image size is >> + * limited in the configuration. >> + * >> + * @throws Exception if an exception occurs while running the test >> + */ >> + @Test >> + public void testGetImageURLWhenBothWidthAndHeightAreUnspecifiedAndImageSizeIsLimited() throws Exception >> + { >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getAttachmentURL("", ""); >> + will(returnValue("")); >> + oneOf(configuration).isIncludeImageDimensionsInImageURL(); >> + will(returnValue(true)); >> + oneOf(configuration).getImageWidthLimit(); >> + will(returnValue(200)); >> + oneOf(configuration).getImageHeightLimit(); >> + will(returnValue(170)); >> + } >> + }); >> + Map<String, String> parameters = Collections.emptyMap(); >> + Assert.assertEquals("?width=200&height=170&keepAspectRatio=true", wikiModel.getImageURL("", "", parameters)); >> + } >> + >> + /** >> + * Tests that the proper image URL is generated when both the width and the height are unspecified and only the >> + * image width is limited in the configuration. >> + * >> + * @throws Exception if an exception occurs while running the test >> + */ >> + @Test >> + public void testGetImageURLWhenBothWidthAndHeightAreUnspecifiedAndOnlyImageWidthIsLimited() throws Exception >> + { >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getAttachmentURL("", ""); >> + will(returnValue("")); >> + oneOf(configuration).isIncludeImageDimensionsInImageURL(); >> + will(returnValue(true)); >> + oneOf(configuration).getImageWidthLimit(); >> + will(returnValue(25)); >> + oneOf(configuration).getImageHeightLimit(); >> + will(returnValue(-1)); >> + } >> + }); >> + Map<String, String> parameters = new HashMap<String, String>(); >> + parameters.put("width", "45%"); >> + parameters.put("style", "height:10em"); >> + Assert.assertEquals("?width=25", wikiModel.getImageURL("", "", parameters)); >> + } >> + >> + /** >> + * Tests that the proper image URL is generated when both the width and the height are unspecified and the image >> + * size is not limited in the configuration. >> + * >> + * @throws Exception if an exception occurs while running the test >> + */ >> + @Test >> + public void testGetImageURLWhenBothWidthAndHeightAreUnspecifiedAndImageSizeIsNotLimited() throws Exception >> + { >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getAttachmentURL("", ""); >> + will(returnValue("")); >> + oneOf(configuration).isIncludeImageDimensionsInImageURL(); >> + will(returnValue(true)); >> + oneOf(configuration).getImageWidthLimit(); >> + will(returnValue(-1)); >> + oneOf(configuration).getImageHeightLimit(); >> + will(returnValue(-1)); >> + } >> + }); >> + Map<String, String> parameters = new HashMap<String, String>(); >> + parameters.put("style", "bad CSS declaration"); >> + Assert.assertEquals("", wikiModel.getImageURL("", "", parameters)); >> + } >> + >> + /** >> + * Tests that the proper image URL is generated when the attachment URL has a fragment identifier. >> + * >> + * @throws Exception if an exception occurs while running the test >> + */ >> + @Test >> + public void testGetImageURLWhenAttachmentURLHasFragmentIdentifier() throws Exception >> + { >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getAttachmentURL("", ""); >> + will(returnValue("test#fragment")); >> + oneOf(configuration).isIncludeImageDimensionsInImageURL(); >> + will(returnValue(true)); >> + } >> + }); >> + Map<String, String> parameters = new HashMap<String, String>(); >> + parameters.put("width", "23"); >> + Assert.assertEquals("test?width=23#fragment", wikiModel.getImageURL("", "", parameters)); >> + } >> + >> + /** >> + * Tests that the proper image URL is generated when the attachment URL has a query string and a fragment >> + * identifier. >> + * >> + * @throws Exception if an exception occurs while running the test >> + */ >> + @Test >> + public void testGetImageURLWhenAttachmentURLHasQueryStringAndFragmentIdentifier() throws Exception >> + { >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getAttachmentURL("", ""); >> + will(returnValue("test?param=value#fragment")); >> + oneOf(configuration).isIncludeImageDimensionsInImageURL(); >> + will(returnValue(true)); >> + } >> + }); >> + Map<String, String> parameters = new HashMap<String, String>(); >> + parameters.put("height", "17"); >> + Assert.assertEquals("test?param=value&height=17#fragment", wikiModel.getImageURL("", "", parameters)); >> + } >> + >> + /** >> + * Tests that the proper image URL is generated when both the style and the dimension parameters are specified. >> + * >> + * @throws Exception if an exception occurs while running the test >> + */ >> + @Test >> + public void testGetImageURLWhenBothStyleAndDimensionParametersAreSpecified() throws Exception >> + { >> + final DocumentAccessBridge documentAccessBridge = getComponentManager().lookup(DocumentAccessBridge.class); >> + final RenderingConfiguration configuration = getComponentManager().lookup(RenderingConfiguration.class); >> + getMockery().checking(new Expectations() >> + { >> + { >> + oneOf(documentAccessBridge).getAttachmentURL("", ""); >> + will(returnValue("")); >> + oneOf(configuration).isIncludeImageDimensionsInImageURL(); >> + will(returnValue(true)); >> + } >> + }); >> + Map<String, String> parameters = new HashMap<String, String>(); >> + parameters.put("height", "46"); >> + parameters.put("width", "101px"); >> + parameters.put("style", "width: 20%; height:75px"); >> + // Note that the style parameter take precedence over the dimension parameters and the width is actually 20% but >> + // we can't use it for resizing the image on the server side so it's omitted from the query string. >> + Assert.assertEquals("?height=75", wikiModel.getImageURL("", "", parameters)); >> + } >> } > > _______________________________________________ > devs mailing list > devs(a)xwiki.org > http://lists.xwiki.org/mailman/listinfo/devs > -- Thomas Mortagne

15 years

1
0
0 0

[xwiki-devs] Help--Issue building JspWikiImporter Class

by Jins Jacob

Hi All, I am trying to convert JSPwiki pages to Xwiki pages.For that i used the code which is in the link http://code.xwiki.org/xwiki/bin/view/Extensions/JSPWikiToXWikiExtension .All the needed jar files had been added to the classpath and try to build this java class.Then it showed the following error Starting up background thread: JSPWiki Lucene Indexer. Starting up background thread: WatchDog for 'Wiki'. com.ecyrd.jspwiki.InternalWikiException: java.io.IOException: Unable to find web.xml for processing. at com.ecyrd.jspwiki.auth.authorize.WebContainerAuthorizer.initialize(WebContainerAuthorizer.java:127) at com.ecyrd.jspwiki.auth.AuthorizationManager.initialize(AuthorizationManager.java:404) at com.ecyrd.jspwiki.WikiEngine.initialize(WikiEngine.java:559) at com.ecyrd.jspwiki.WikiEngine.<init>(WikiEngine.java:369) at JspWikiImporter.export(JspWikiImporter.java:76) at JspWikiImporter.main(JspWikiImporter.java:50) com.ecyrd.jspwiki.WikiException: Failed to start managers: java.io.IOException: Unable to find web.xml for processing. at com.ecyrd.jspwiki.WikiEngine.initialize(WikiEngine.java:610) at com.ecyrd.jspwiki.WikiEngine.<init>(WikiEngine.java:369) at JspWikiImporter.export(JspWikiImporter.java:76) at JspWikiImporter.main(JspWikiImporter.java:50) Caused by: com.ecyrd.jspwiki.InternalWikiException: java.io.IOException: Unable to find web.xml for processing. at com.ecyrd.jspwiki.auth.authorize.WebContainerAuthorizer.initialize(WebContainerAuthorizer.java:127) at com.ecyrd.jspwiki.auth.AuthorizationManager.initialize(AuthorizationManager.java:404) at com.ecyrd.jspwiki.WikiEngine.initialize(WikiEngine.java:559) ... 3 more I added a dummy web.xml in WEB-INF directory.But the same issue occurs.What is to be done to rectify this issue and what are the essential components that are needed in the said web.xml? Please help me. If anybody have any idea please let me know. Thanks, Jins Jacob

15 years

1
0
0 0

[xwiki-devs] [proposal] Develop form, field, and validation constraint classes and abandon the practice of storing form information in the class.

by Caleb James DeLisle

Hi, I would like to propose a change in direction. I think the practice of marrying form fields to class properties should be discontinued. Including form information in the class property makes certain jobs easier but XObjects do not always represent forms and marrying something as high level as forms with something as low level as objects makes code maintenance exceedingly difficult, not to mention the general ugliness of code in xwiki-core which generates HTML. I would like to start development of 3 new XWikiDocuments: XWiki.FormClass, XWiki.FormFieldClass and XWiki.FormFieldConstraintClass. Much of the code in XWiki.Registration will be ported to these classes. The fields will be validated twice, once at the client side with LiveValidation, and once at the server side. I would then like to deprecate the use of the display related settings in the class editor and eventually remove them from the class editor entirely. I envision a form being made and fields added similar to the way we add objects to a document. Likewise, each field will be defined by selecting a display type from a predefined list then adding validation constraint objects to the field. I have not started development on this and I am sure there will be plenty more to discuss later. What I want to know is whether people think this is the right direction. WDYT? Caleb

15 years

4
4
0 0

[xwiki-devs] Issue building JspWikiImporter Class

by Jins Jacob

Hi All, I am trying to convert JSPwiki pages to Xwiki pages.For that i used the code which is in the link http://code.xwiki.org/xwiki/bin/view/Extensions/JSPWikiToXWikiExtension.All the needed jar files had been added to the classpath and try to build this java class.Then it showed the following error * * * Starting up background thread: JSPWiki Lucene Indexer. Starting up background thread: WatchDog for 'Wiki'. com.ecyrd.jspwiki.InternalWikiException: java.io.IOException: Unable to find web.xml for processing. at com.ecyrd.jspwiki.auth.authorize.WebContainerAuthorizer.initialize(WebContainerAuthorizer.java:127) at com.ecyrd.jspwiki.auth.AuthorizationManager.initialize(AuthorizationManager.java:404) at com.ecyrd.jspwiki.WikiEngine.initialize(WikiEngine.java:559) at com.ecyrd.jspwiki.WikiEngine.<init>(WikiEngine.java:369) at JspWikiImporter.export(JspWikiImporter.java:76) at JspWikiImporter.main(JspWikiImporter.java:50) com.ecyrd.jspwiki.WikiException: Failed to start managers: java.io.IOException: Unable to find web.xml for processing. at com.ecyrd.jspwiki.WikiEngine.initialize(WikiEngine.java:610) at com.ecyrd.jspwiki.WikiEngine.<init>(WikiEngine.java:369) at JspWikiImporter.export(JspWikiImporter.java:76) at JspWikiImporter.main(JspWikiImporter.java:50) Caused by: com.ecyrd.jspwiki.InternalWikiException: java.io.IOException: Unable to find web.xml for processing. at com.ecyrd.jspwiki.auth.authorize.WebContainerAuthorizer.initialize(WebContainerAuthorizer.java:127) at com.ecyrd.jspwiki.auth.AuthorizationManager.initialize(AuthorizationManager.java:404) at com.ecyrd.jspwiki.WikiEngine.initialize(WikiEngine.java:559) ... 3 more* ** I added a dummy web.xml in WEB-INF directory.But the same issue occurs.What is to be done to rectify this issue and what are the essential components that are needed in the said web.xml? Please help me. Thanks, Jins Jacob

15 years

1
0
0 0

Re: [xwiki-devs] [Proposal][UX] Recent Changes Improvements

by Jerome Velociter

----- "Fabio Mancinelli" <fabio.mancinelli(a)xwiki.com> wrote: > On 08/30/2010 05:46 PM, Ecaterina Valica wrote: > > Hi Fabio, > > > > This means I am iterating and I gather feedback and depending on > that > > feedback we must agree on a version. > > > Ok. I thought that "old" meant "superseded by" :) > > > On a wiki platform, what do you think is more relevant: the user or > page > > centric approach? > > What are the things that make this version (the page centric one) > not to fit > > the Wiki 3.0 specification? > > > What is missing basically boils down to the "Share your thoughts" box Yes, user updates should be mixed in the stream ultimately, I think. > > and the type of information we can convey with it. > > The "user centric" view, imho, is also useful because a "user" could > also be an application that sends events (e.g., meeting or deadlines > reminders, etc.) that can be shown in a more natural way in a user > centric view. Basically even applications might share their thoughts > through the activity stream ;) I completely agree with the last sentence. BUT I think it's not incompatible with a document/user mixed-approched (I don't think either full document-centric or full user-centric can embrace the versatility of XWiki) I see two types of entries : * Document updates. Basically what is currently in latest proposal (though I would add little user pictures here as well, even before the click). This is KM, collaboration, etc. * Users/applications updates. Those are less structured, more a-la facebook. I see something like http://incubator.myxwiki.org/xwiki/bin/view/Improvements/ActivityStreamMixed (raw picture, would have to be refined) (Note I also have right-padded a bit more the comment/annotation/pencil/etc icons that are on top of user photos, I think they were a bit too masked by them) Jerome. > > > Jerome missed the avatars, so if you did too, please try to expand > the items > > in the prototype (when you hover the items an arrow appears in the > right > > part suggesting that the item can be expanded ). > > > Yes I missed them too, but then I saw it. > Still the previously mentioned types of information are not available > > anymore. > > Thanks, > Fabio > _______________________________________________ > devs mailing list > devs(a)xwiki.org > http://lists.xwiki.org/mailman/listinfo/devs

15 years

5
11
0 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

xwiki-devs