Make string -> function link a bit less agressive (no class methods/properties)
>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #1426118. --> Reported by: [jhodgdon](https://www.drupal.org/user/155601) >>> <p>In <span class="drupalorg-gitlab-issue-link project-issue-status-info project-issue-status-7"><a href="https://www.drupal.org/project/api/issues/294218" title="Status: Closed (fixed)">#294218: Turn hook names in theme() and module_invoke() into links</a></span> and <span class="drupalorg-gitlab-issue-link project-issue-status-info project-issue-status-7"><a href="https://www.drupal.org/project/api/issues/745204" title="Status: Closed (fixed)">#745204: Turn strings that are callback function names into function links</a></span> we are turning various strings in code into links.</p> <p>I think we need to change the order of this somewhat and adjust it.</p> <p>Take a look at this screenshot for instance. 'name' shouldn't really be a link -- it is coming up because there is a class property or method called 'name', and I don't think we should really make 'name' into a link because of that -- only should be done for bare functions not part of classes.</p> <p>And the module_invoke() link for 'install' is not actually linking to hook_install() [which would be appropriate here] but is instead linking to Updater::install -- two ways to fix this:<br> a) check for the module_invoke(..., 'string') pattern before the generic 'string' pattern<br> b) don't do class stuff (fix for the above).</p> <p>So I think both (a) and (b) should be done before we release this code into the wild.</p>
issue