Port to Drupal 7. Consider using Meta Tags (not ready?), Nodewords or breaking the dependency and not using either. TBD.

CommentFileSizeAuthor
#14 1208366-agls-drupal-7.patch18.68 KBnick_schuch
#8 agls_d7.zip3.08 KBrooby

Comments

Bevan’s picture

rooby’s picture

Status: Needs work » Active

It should rely on the Meta tags module, which looks ready enough now.

rooby’s picture

Since AGLS is an extension of dublin core I think this module should only implement tags that are different from dublin core.

There is already plans for dublin core support for metatags, see:
#1491616: Add new meta tag: Dublin Core
#1278146-13: [META] Ensure all meta tags are supported (no patches, just planning)

So all this module would need to provide are the tags:
* Function
* Availability
* Audience

Then possibly also alter the dublin core tags to add qualifiers like language & scheme.

rooby’s picture

However, xml:lang/lang & scheme attributes probably should also be handled in the parent metatag module if possible.

rooby’s picture

Some related issues for metatag:
#1500396: Support encoding schemes (scheme attribute)
#1282620: Integration with i18n for per-language defaults - The xml:lang/lang attribute may not end up being part of that issue.

rooby’s picture

Actually, the audience tag is part of dublin core too, it is just part of qualified dublin core, not simple dublin core.

So the metatags module probably should also support that.

rooby’s picture

Assigned: Bevan » rooby

I'm going to do this shortly.

rooby’s picture

Status: Active » Needs review
StatusFileSize
new3.08 KB

Here is the first cut of the Druapl 7 version.
It is essentially a complete rewrite from scratch, since so much has changed since drupal 6.

It is a first cut only because it includes the required dublin core tags, which will soon be part of the meta tags module, so will need to be removed from here.
If we need to modify them in some way we can alter the meta tag ones.

There are also some sub-terms I have not added yet, for example aglsterms:act (previously agls:mandate:act).

It is basically just what I needed right now for a website that needs it right now.

I will probably want to add some more useful links to the readme and some scheme information to the descriptions of tags in the admin UI.

To the module maintainer:
If you are willing to add me as a committer I would be happy to maintain the drupal 7 version.

rdeboer’s picture

This is great, rooby!
Need this right now! Keep up the great work!

rooby’s picture

Yeah, it isn't fully finished, but like you I needed it right now.
It is enough to pass an AGLS requirement.

nick_schuch’s picture

I have tested the module listed in comment #8. Module was able to install and run the module as intended. I believe there is definitely enough here for a 7.x-1.x branch.

I will setup the branch shortly.

rooby’s picture

Thanks.
I have been using it on a live site with no drama.

However, I think a proper release should await the outcome of #1491616: Add new meta tag: Dublin Core, so hopefully all the DC stuff can be left out.

nick_schuch’s picture

In relation to comment #12.

I completely agree.

nick_schuch’s picture

StatusFileSize
new18.68 KB

After a recent project I have made the following changes to bring this module inline with WCAG 2.0 standards. I have also taken into account the wording from the current Metatags issue queue (#12). This will provide (for now) a method for people to get AGLS up and going quickly.

This patch is based against the current state of the 7.x-1.x branch.

kim.pepper’s picture

Commited #14 to 7.x-1.x

kim.pepper’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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