Taxonomy integration

Share a patch, please.

Subscribe.

#1067120: Missing hook_taxonomy_term_view() or hook_entity_view() when viewing a term

Issue #1067120: Missing hook_taxonomy_term_view() or hook_entity_view() when viewing a term is solved for 7.x, the patch is included in 7.17.
Does this module need some change to work with terms now that the include the standard entity hooks or it just works now?

Thanks for this great module!

Nope, this may need a lot of work.

Hi guys,

In my work i need integration with taxonomy, bean and field_collection.

I created big patch for taxonomy part, please review it. I suggest exclude each functional part in different submodules(in future for existing parts too). Check how i did in my patch. I made ​​everything as well as in the main module.

To use not forget enable "linkchecker_taxonomy" module.

Again...

Looks really good to me, but can you add some test, please?

I can, but then later, because this work will be only in free time

Well, are we not all working for free? At least me.

We should make sure that this will not break some days without getting noticed.

Well, are we not all working for free? At least me.

Yes of cource, i meant that i needed this functional in my work.

We should make sure that this will not break some days without getting noticed.

Yes, of course. We can wait other reviews and in any case will be better also write tests. But i can't write it right now.

So, i split previous patch. Was appended few hooks to get independent of modules.

hass, please review this patch

After i will append patches for taxonomy and beans modules.

Patch for taxonomy, again(previous patch WRONG).

Should be applied after integration-independent-modules-1390872-15.patch will be implemented!

Appended patch for bean integration.
Small fixes in taxonomy integration.

Not forget please, that these patches should be implemented after integration-independent-modules-1390872-15.patch

Small fixes in delete function.

Not forget please, that these patches should be implemented after integration-independent-modules-1390872-15.patch

Could we split this into seperate issues, please?

Hi Alexander,

Why you showed me old patches?
Should i move taxonomy and beans in other tasks? And in this task we keep "integration-independent-modules" ?

Let's make this the taxonomy case here and move bean to an extra case. I only hided outdated patches. See issue summary.

follow #2127731: Bean compatibility

Have you seen #2060243: Split linkchecker_scan_nodetypes as content type specific? I plan to move setting to conent type settings. Maybe we can add the vertical tab to taxonomy settings page, too.

Ok, i will prepare new patch, for taxonomy only (seems good idea split settings)

Applying #15 and then #19 appears to be working beautifully for taxonomy integration. Thanks!

#15 has a typo in the API name, linkcheker_main_table.

Hi @hass
(/) corrected the typo
(/) replaced module_invoke_all with Drupal::moduleHandler()->invokeAll
(/) replaced depricated l() with Link::fromTextAndUrl and Url::fromUri

Please do not waste your time on new features until the basic port is completed.

This already fixed by new logic of link extraction for D8.