>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #1863780. --> Reported by: [dydave](https://www.drupal.org/user/467284) >>> <p>Hi guys,</p> <p>Thanks very much for this great module and all the nice features it adds on top of <a href="http://drupal.org/project/facetapi">Facet API</a>.</p> <blockquote><p> I've been working recently with the <a href="http://drupal.org/project/metatag">Meta tags</a> module along with a setup on search_api, views, facetapi, search_api_db, etc... with a standard page setup and was wondering about the possibility of integrating the tokens provided by facetapi with the <a href="http://drupal.org/sandbox/ribakker/1356928">Meta tags views</a> module (extending Meta tags).<br> <strong>The idea here</strong> is to be able to display values provided by the facetapi tokens in the meta tags and other head markup elements on search enabled pages.</p> <p>After looking into <a href="http://drupal.org/project/facetapi_bonus">Facetapi Bonus</a>, which brought me to <a href="http://drupal.org/project/page_title">Page Title</a>, I came up with a patch that seems to have got the maintainer's attention, requesting further documentation (See: <span class="drupalorg-gitlab-issue-link project-issue-status-info project-issue-status-7"><a href="https://www.drupal.org/project/metatag/issues/1830952" title="Status: Closed (fixed)">#1830952: Introduce two new hooks to allow token types and patterns alteration</a></span>).<br> <strong>In short</strong>, it's about introducing 2 new hooks in metatag module's api, to allow exposing tokens to metatag patterns and altering patterns before replacement. </p></blockquote> <p>More information at: <span class="drupalorg-gitlab-issue-link project-issue-status-info project-issue-status-13"><a href="https://www.drupal.org/project/facetapi/issues/1856436" title="Status: Needs work">#1856436: Facetapi Tokens: Attempt to initialize tokens based on current active searchers.</a></span>.</p> <p><strong>In short:</strong> I have posted a feature request to add two hooks in the metatag module API that would <strong>allow overriding replacements and providing default Facet API token values</strong> in a similar way the <a href="http://drupal.org/project/facetapi_bonus">Facetapi Bonus</a> does it currently for the <a href="http://drupal.org/project/page_title">Page Title</a> module.<br> See the metatag API hooks request: <span class="drupalorg-gitlab-issue-link project-issue-status-info project-issue-status-7"><a href="https://www.drupal.org/project/metatag/issues/1830952" title="Status: Closed (fixed)">#1830952: Introduce two new hooks to allow token types and patterns alteration</a></span> (Contains more details about the implementation based on what was done for the <a href="http://drupal.org/project/page_title">Page Title</a> module).<br> See also: <a href="http://drupalcode.org/project/facetapi_bonus.git/blob/1f2e720:/facetapi_bonus.page_title.inc">facetapi_bonus.page_title.inc</a></p> <p><strong>Please find attached to this ticket a patch tested against facetapi_bonus-7.x-1.1+7-dev (2012-10-22), for integrating <a href="http://drupal.org/project/facetapi">Facet API</a> tokens with the <a href="http://drupal.org/project/metatag">Meta tags</a> API token hooks<strong>, file: facetapi_bonus.metatag.inc which implements:</strong></strong></p> <ul> <li>hook_metatag_token_types_alter</li> <li>hook_metatag_pattern_alter</li> </ul> <p>[File attached as: <a href="http://drupal.org/files/facetapi_bonus-metatags-facetapi-tokens-support-0.patch">facetapi_bonus-metatags-facetapi-tokens-support-0.patch</a>]</p> <p><br></p> <h3>Additional Comments:</h3> <p>Implementation of <code>hook_metatag_pattern_alter</code> could potentially be avoided if the following feature requests could be accepted:</p> <ol> <li>Default values could be provided to the $data array in facetapi_tokens, see: <strong><span class="drupalorg-gitlab-issue-link project-issue-status-info project-issue-status-13"><a href="https://www.drupal.org/project/facetapi/issues/1856436" title="Status: Needs work">#1856436: Facetapi Tokens: Attempt to initialize tokens based on current active searchers.</a></span></strong><br> This would allow removing some of the logic related with facetapi token values default $data initialization, in particular for <strong>facetapi_results</strong>, <strong>facetapi_active</strong> and <strong>facetapi_facet</strong> tokens.<br> Logic for <code>list&lt;[facetapi_active:facet-label]&gt;</code> (for example) is the next point.</li> <li><strong>facetapi_tokens</strong> could provide a <em>facetapi_active_list</em> token for a list of all active facets.<br> In a way, this would come to merging into facetapi_tokens the logic from <code>facetapi_bonus_metatag_pattern_alter</code>, for example: <pre>&nbsp;&nbsp;&nbsp; // If we meet list&lt;...[facetapi_active:XXX]...&gt; string we replace it with<br>&nbsp;&nbsp;&nbsp; // coma separated list of tokenized string.<br>&nbsp;&nbsp;&nbsp; // So list&lt;[facetapi_active:facet-label]: [facetapi_active:active-value]&gt;<br>&nbsp;&nbsp;&nbsp; // will be replaced with "type: node, author: admin" string if type and<br>&nbsp;&nbsp;&nbsp; // author facets active.</pre><p> I was just thinking it would be useful for other modules to be able to provide this <strong>active facet list</strong> token, directly visible from the backend interface, for example, and not in code/comment notes or on the project page.<br> In any case, this should probably be discussed in another ticket, but I wanted to get that documented here as well.</p></li> </ol> <p>If these <strong>feature requests</strong> could be integrated to <em><a href="http://drupalcode.org/project/facetapi.git/blob/b53df46:/facetapi.tokens.inc#l124">facetapi_tokens</a></em>, in file <strong><a href="http://drupalcode.org/project/facetapi.git/blob/b53df46:/facetapi.tokens.inc">facetapi.tokens.inc</a></strong>, then it would <strong>eliminate the need for token replacements</strong> for the <a href="http://drupal.org/project/page_title">Page Title</a> or <a href="http://drupal.org/project/metatag">Meta tags</a> (<code>hook_metatag_pattern_alter</code> could be removed) modules, <strong>though there would still be a dependency</strong> for <code>hook_metatag_token_types_alter</code>.</p> <p>Could you please let me know if I overlooked or missed anything in the current <a href="http://drupal.org/project/facetapi">Facet API</a>, <a href="http://drupal.org/project/metatag">Meta tags</a> API or <a href="http://drupal.org/project/facetapi_bonus">Facetapi Bonus</a> modules, in terms of whether this would be the correct implementation or not?</p> <p>Please let me know if you would have any questions on any points/code/aspects mentioned in this ticket, I would surely be glad to provide more information.</p> <p>I would greatly appreciate if you could take a bit of time to look into the <a href="http://drupal.org/files/facetapi_bonus-metatags-facetapi-tokens-support-0.patch">patch</a> and give me your feedback/opinion on this <strong>feature request</strong>.</p> <p>Any feedback, testing, changes, recommendations would be highly appreciated.<br> Thanks in advance.</p> > Related issue: [Issue #1830952](https://www.drupal.org/node/1830952) > Related issue: [Issue #1856436](https://www.drupal.org/node/1856436) >>> [!note] Pre-migration files - [facetapi\_bonus-metatags-facetapi-tokens-support-0.patch](https://www.drupal.org/files/facetapi_bonus-metatags-facetapi-tokens-support-0.patch) >>>