broken with chameleon (five.pt)

[simahawk]

I'm testing eea.daviz on plone 4.2.3 and chameleon (five.pt). The view template is broken is due to a missing namespace (ex:) declaration in eea/app/visualization/facets/list/view.pt. I fixed temporarely by wrapping the template in this declaration

'... xmlns:ex="http://daviz.eea.europa.eu/namespaces/ex">'

but it needs a better namespace for sure :)

[simahawk]

same happen in eea/app/visualization/browser/zpt/view.pt

[demarant]

Hi Simahawk.

We do not use chameleon at EEA.

However, if you use it, then you are able to test it.
Please, go ahead and do the fixes and send us a pull request to the collective fork.

Regarding the namespace, "ex:" stands for the Simile Exhibit namespace .... which I think already exists, I think we could add this one:
xmlns:ex="http://simile.mit.edu/2006/11/exhibit#"

reference: http://simile.mit.edu/wiki/Exhibit/How_to_include_the_Exhibit_API

[simahawk]

Ciao :) ok tnx I'll use that.

[alecghica]

Hi @simahawk, can you please (re)do the pull request on /collective/eea.app.visualization as we keep this repo in auto-sync with the one from /collective where all Plone contributors have rights. We can do the manual merge but will be more easy for you just to re-do the pull request on collective. We will add a HowTo contribute to this package just to be clear for everyone. Unfortunately same will apply for the other 3 pull requests you made on eea.googlecharts, eea.exhibit and eea.app.visualization:

• [imp] use a more sane jquery src load and make chameleon happy eea.googlecharts#1
• [fix #2] add namespaces eea.exhibit#3
• [add] exhibit namespaces makes chameleon happy. addresses #1 #2

[simahawk]

done ;)

[alecghica]

Great, thx! :)

[alecghica]

Fixed under collective#4