Are there any plans for D7 version ?
I would be very interested.
Thank you for any information.

Comments

nonsie’s picture

Status: Active » Postponed

Not at the moment as Metatag is completely different from Nodewords. Should an opportunity present, I'd love to do it and/or maintain it. There is currently no upgrade path for Metatag from Nodewords (D6). I'm going to wait until there is one before proceeding.
As always - patches are welcome though.

damienmckenna’s picture

Title: port to D7 » Port Domain Metatags to D7
netikseo’s picture

Any updates? It's very important to have different front page metatags on each subdomain...

Thanks!

damienmckenna’s picture

Status: Postponed » Active

Nothing current, just need time to get to it or a project that needs it. If anyone else happens to have some time we'd greatly appreciate patches :)

summit’s picture

Hi,
Would love to see Meta Tags support Domain Access module.
My use case is being able to have different domain tokens like terms, nodes to be inserted in the domain meta tags so that the meta tags show also that you are on a different domain with different title, key words and description.
May be through the entity way? Something like http://drupal.org/node/1688286 but than not for languages, but for domains?
Greetings, Martijn

bcobin’s picture

See http://drupal.org/node/1788292 - in short, @agentrickard states that:

hook_domain_batch() in a custom module is the likely solution.

Perhaps someone with coding skills would want to take a crack at it...

nonsie’s picture

The code required to get it working for DA in D7 is not that much different from D6. What has changed and will be time consuming is Nodewords/Metatag part.

damienmckenna’s picture

Someone has already built this and I'm discussing with them to merge their work into this module. More later.

artofeclipse’s picture

Sorry for the delay, I'm currently working on D7 port for this project and merging domains metatag and it will be in the next couple week.

summit’s picture

HI Ma'moun, great to hear.
Looking forward to it! Would be great before Christmas :)
greetings, Martijn

remkovdz’s picture

I need to set different meta descriptions (each for a domain) per node. Is that what this module (when ported) intents to do or will it only work for the frontpage?

damienmckenna’s picture

remkovdz’s picture

Hi, I'm still curious if, when this module is ported to D7, this will only work for the front page? Or will it for example be possible to set default Meta Tags for each domain?

My use case:
- 3 domains
- Same products on all domains, but with very different target audience
- Audiences: B2B, B2C informational, B2C transactional (webshop)

For instance, I'd like the B2B Meta Tags always to end with something like "Contact us for more information", and the B2C webshop with something like "Buy now".

Will this be possible?

damienmckenna’s picture

@artofeclipse: Any update on your progress?

artofeclipse’s picture

@DamienMcKenna I'm working on it right now, sorry for the delay on this issue, I will push my code to 7.x branch and hopefully we will get it working soon.

damienmckenna’s picture

@artofeclipse: no worries :) Thanks for the update!

artofeclipse’s picture

Assigned: Unassigned » artofeclipse

@DamienMcKenna I pushed my changes to the branch please review my code and test it, if there is any change I must do just inform me about it :) and if you have time maybe we can have a skype call to discuss the changes/new feature or a better implementation.

remkovdz’s picture

Can I test D7 version somewhere? And also, please have a look at #13, thanks a lot.

nonsie’s picture

StatusFileSize
new3.9 KB
new3.64 KB

@artofeclipse - I took a look at the code you committed. Attached is a patch to make it better.
Some notes:
- run the code through Coder, it gives you the obvious missing pieces
- I don't understand the need for ksort in _domains_metatag_get_active_domains as the keys are incremental to begin with. The domains are ordered by weight due to domain_id_list() which is called by domain_domains().
- you should never use a variable inside of t() function, use a placeholder instead (see http://api.drupal.org/api/drupal/includes%21bootstrap.inc/function/t/7)
- don't create variables you are not going to use

@DamienMcKenna - anything from Metatag that should be revised?

nonsie’s picture

StatusFileSize
new3.9 KB

Patch attach fail. This is the right one.

artofeclipse’s picture

@nonsie, Thanks for the patch and your comments, patch applied and pushed to the codebase

netsensei’s picture

Issue summary: View changes

Is it possible to create a D7 release of the dev branch? Or potentially even a D7 stable release if this is ready to go?

Thx!

yogeshchaugule8’s picture

Hi Nonsie,

Can we please add Drupal 7 Module link on this module page? So its easy to find D7 module.

damienmckenna’s picture

Status: Active » Fixed

I've released the dev version, lets work from there to finish off the module.

Status: Fixed » Closed (fixed)

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