Add an 'exclude' xpath option

Looks like we need to add hook_update to add new option for existing installations.

#4 worked for me. Updated patch to fix a formatting issue and add an example xpath to field description to give users a good default for ignoring anything with a class of toc-ignore.

@vladimiraus, don't think we need a hook_update() for this. I added this patch to an existing install and it worked great; no change to functionality unless I manually add an xpath.

Would be nice to get this included in a new release. I moved a site from drupal/tocify to this module because tocify relied on JS to build the TOC client-side and that had inherent accessibility issues. When doing a module comparison this module only lacked the ability to ignore certain DOM elements, which this patch fixes.

Thanks @smulvih2 for the contribution. Couple of things to fix:

• hook_update for current installation to include header_exclude_xpath to all existing TOCs
• change new \DOMXPath($dom); to new \DOMXPath($dom);

@vladimiraus, I don't think a hook_update() is needed here. Without a hook_update() the header_exclude_xpath line is added to existing TOC config when a TOC is saved. Otherwise if the header_exclude_xpath line is missing from TOC config it doesn't cause an issue. The options array is protected so not sure how to update this, please advise.

Also I tried removing the slash before DOMXPath($dom); which is what I think you were trying to say in your last comment, but this causes an error, so working fine as is.

New patch with hook_update() to add header_exclude_xpath to all existing TOCs.

@VladimirAus any chance you can review this and get into a new release? I'm using this patch on a few projects and its working great.

Thanks for your commit @smulvih2.
Give me few days...

As I am using this functionality without this patch; safe to say this has been committed and this should be marked fixed/closed.

Also, wouldn't //*[contains(@class,"toc-ignore")]//* have been a better default?

@liquidcms if you are using WxT this patch is included in the distro which is why it is working for you. Still waiting for this patch to be rolled into latest version of toc_api.

Thank you. Commited.

@VladimirAus,
thank you!