-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix Canonical URL updater form #118
Conversation
A new upgrade step is also provided to update the objects_provides catalog index.
We need to find out where to open an issue about this.
|
||
def test_upgrade_to_3046_registrations(self): | ||
version = self.setup.getLastVersionForProfile(self.profile_id)[0] | ||
self.assertGreaterEqual(int(version), int(self.to_version)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tcurvelo I have no idea why this is failing, I think is something from another test; do you mind to dig on it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I already found the issue and fixed it: 78bc1e1
@@ -42,11 +30,12 @@ class ICanonicalURLUpdater(model.Schema): | |||
constraint=validate_canonical_domain, | |||
) | |||
|
|||
created_before = schema.Date( | |||
published_before = schema.Date( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I miss the 'date' name in this attribute, how about to change it to 'published_before_date'?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added some comments
I opened this thread on the community forum to find a solution to the error in Plone 5.1: https://community.plone.org/t/strange-catalog-behavior-on-plone-5-1/4582?u=hvelarde |
@@ -23,21 +24,31 @@ def setUp(self): | |||
self.portal = self.layer['portal'] | |||
self.request = self.layer['request'] | |||
enable_social_media_behavior() | |||
self.setup_content() | |||
api.portal.set_registry_record( | |||
name='canonical_domain', value='https://example.org', interface=ISocialLikeSettings) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought canonical_domain
becomes old_canonical_domain
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only in the form.
A new upgrade step is also provided to update the
objects_provides
catalog index.WIP
fixes: #115