[WIP] - Fix location accessor

cekk commented 5 years ago

Location accessor for events didn't return the default "location" field value, if set.

Now the accessor looks for venue field, and if not set, looks into the default location field.

I've also added a test for this feature and setup travis conf.

cekk commented 5 years ago

@thet i've found two problems:

behavior fields for reference a location and an organizer were not correctly serialized by plone.restapi. I switched them to be real relation fields and changed their names.
Control panel breaks on save. This happens also on a Plone 5.1.5. Is it really used? Can we remove it and all defaults?

These two problems are related to p.a.vocabularies. The first one happens only with newer p.a.vocabulary and restapi because i think that something has changed in CatalogVocabulary.

Are these changes ok for you?

cekk commented 5 years ago

Another thing: i've made an upgrade-step to fix old fields, but it doesn't show up in products control panel (maybe because there are different profiles).

From portal_setup everything works well

cekk commented 4 years ago

@thet any news about this pr?

collective / collective.venue

[WIP] - Fix location accessor #16