Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna created an issue. See original summary.

DamienMcKenna’s picture

thejimbirch’s picture

Assigned: Unassigned » thejimbirch

Working on this.

thejimbirch’s picture

Patch attached that adds the refresh meta tag.

Steps to validate:
Apply the patch
Edit a metatag field or default and add a number or a number and a url (5; url=http://example.com/)

Save and view the source of the rendered page

The browser will redirect the page after the amount of time specified.

cindytwilliams’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
47.72 KB
36.84 KB

Patch applied cleanly. The refresh field was displayed on the metatag admin configuration screen. When a value was entered into the field, the refresh metatag appeared in the source of the rendered page, and the browser redirected to the page after the amount of time specified. Marking RTBC.

DamienMcKenna’s picture

Status: Reviewed & tested by the community » Needs work

Could you please also edit the "refresh" line in src/Plugin/migrate/process/d7/MetatagD7Entities.php? Thanks.

thejimbirch’s picture

Status: Needs work » Needs review
FileSize
1.91 KB

New patch attached that adds refresh to the tagsMap function in the MetatagD7Entities class (very cool!)

thejimbirch’s picture

Realized I missed the schema on this one also. New patch attached.

DamienMcKenna’s picture

cindytwilliams’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
282.22 KB

Patch applied cleanly, and added refresh to the tagsMap function in the MetatagD7Entities class and also to the schema. Functionality still works as expected -- when the "Refresh" field is populated it adds a refresh metatag and browser redirects to the specified page. Marking RTBC.

DamienMcKenna’s picture

Status: Reviewed & tested by the community » Fixed

Committed. Thanks everyone!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.