search

datakurre / collective.pfg.dexterity

Dexterity PFG Adapter
2 stars 3 forks source link

cannot choose target folder #7

Closed petri closed 11 years ago

petri commented 11 years ago

On c.p.d 0.60 and Plone 4.2.4 selection of the target folder is broken. The plone dialog allows for selecting the folder, but the information is not updated in the adapter add form. Consequently it is not even possible to add the adapter.

datakurre commented 11 years ago

Thanks. Could you try this with Plone 4.2.5 pre-release?

datakurre commented 11 years ago

That seems to be the same issue, why Travis-build is currently failing: https://saucelabs.com/tests/6d209d81df3042bbab8d6fa0a15aa0a0#

petri commented 11 years ago

Confirmed that the issue remains the same with Plone 4.2.5 pre.

datakurre commented 11 years ago

Ok. I'm afraid that this is caused by some javascript-issue outside c.pfg.dexterity, but I was already going to play more with Sauce Labs -tests during the weekend, so I should be able to figure this out.

petri commented 11 years ago

I think you're probably right. I even remember seeing this kind of issue before; I seem to recall it's got something to do with the selection dialog settings.

datakurre commented 11 years ago

The latest test run is available at: https://saucelabs.com/u/cpfgdexterity

It has been run with Plone 4.2.4 and it shows this issue for every browser.

TODO: Make Travis-CI-matrix run tests for different Plone versions...

datakurre commented 11 years ago

The issues was with archetypes.referencebrowserwidget >= 2.4.15 < 2.4.17

    - 2.4.15 broke non-multi reference selection. Added code to discriminate
      between multi and single widget id. Fixes http://dev.plone.org/ticket/13402
      [smcmahon]

Pin archetypes.referencebrowserwidget = 2.4.17 to fix this issue.

jriveramerla commented 11 years ago

I keep getting this error, I don't think it's solved, does anyone has the same problem ?

datakurre commented 11 years ago

@jriveramerla See: https://github.com/datakurre/collective.pfg.dexterity/issues/7#issuecomment-14973311

This cannot be fixed in this package, but you should use working version of archetypes.referencebrowserwidget.