Problem/Motivation

We use Taxonomy Term Glossary to great effect in providing students with help when it comes to dealing with the extraordinary number of synonyms that we tend to throw around in the Computer Science arena.

Originally we only intended to use to highlighting key terms within content but it soon became apparent that it is also priceless for highlighting terms when looking up the meaning of 'terms' themselves.

For example, in the illustration below, you can see someone the the term 'authentication' (1) being highlighted because it is in the Glossary...

taxonomy_term_glossary_highlight_authentication

When the student clicks through to 'Read more' (2), they are then taken to the term in the modal (3)...

taxonomy_term_glossary_click_through_to_authentication

...and in there, it is very useful to be able to have other terms (e.g. biometrics) highlighted too but the original term itself 'Authentication' should not really be highlighted as it might lead them to go around in circles.

Just to add to this, the same thing applies to Synonyms which we use extensively, synonyms of the current term should not ne highlighted as again they will lead to to go around in circles. In the example below both 2FA and TFA are synonyms of 'Two Factor Authentication'...

taxonomy_term_glossary_click_through_to_two_factor_authentication

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

sirclickalot created an issue. See original summary.

sirclickalot’s picture

mably’s picture

Sounds like an interesting feature to have. Will see what I can do.

mably’s picture

Status: Active » Needs review
StatusFileSize
new61.56 KB

@sirclickalot can you give a try to this issue's MR?

A new option have been added to the Term Glossary configuration form allowing to disable self-referencing of terms:

Self-references config form

sirclickalot’s picture

StatusFileSize
new39.01 KB

@mably,

Very nice indeed!

Does not self-reference anymore (1) but does honour related synonyms (2) and (3)...

taxonomy_term_glossary_fixed

Thank you.

mably’s picture

Are TFA and Biometrics defined as synonyms of the term Authentication ?

If it's the case they shouldn't be rendered as links. It works fine on my side 🤔

Should we merge this as is?

sirclickalot’s picture

No, they are not, 'Authentication' in our case has only one synonym which is 'authenticate'.

I think it's all good and you can merge away.

Thanks.

  • mably committed 196ba6f4 on 4.x
    Issue #3535737 by sirclickalot, mably: Exclude self-referencing of terms
    
mably’s picture

Component: Miscellaneous » Code
Status: Needs review » Fixed

Merged! Thanks.

sirclickalot’s picture

V4.4.0-rc2 tested and verified as A OK with me.

mably’s picture

Great, thanks @sirclickalot!

Status: Fixed » Closed (fixed)

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