- First commit to issue fork.
- 🇪🇸Spain abelass
I know this is closed, but in our project we need to hide this comment for security reasons. So here an updated patch #18 for 4.2.2. For in case it is useful for somebody
Firstly, thanks a lot for this great module, I barely started to play with it but so far it does a fantastic job.
Reasoning behind this feature request:
I get the temptation to advertise the module but some (including me) might think it is not cool at all that to have a module advertisement on their website. I've seen this practice in Joomla! and Wordpress extensions, now it's the first time I see it in the Drupal world.
And AFAIK there isn't a standard way of providing "Generated by" information into the sitemap standard.
So my request is:
Drop the following:
From the generated sitemap.xml:
<!--Generated by the Simple XML Sitemap Drupal module: https://drupal.org/project/simple_sitemap.-->
From the generated sitemap.xsl:
<p>Generated by the <a href="https://www.drupal.org/project/simple_sitemap">Simple XML Sitemap</a> Drupal module.</p>
Or if you disagree with my view at least give the user a choice of whether they want the "Generated by" or not.
To save some of your time I can provide a patch for the dropping of "Generated by" but I guess you will disagree with my view and will not include the patch int the mainstream module. So I don't know how fast I will get familiar enough with the code to provide a patch for the second option.
Just let me know about which path you're leaning towards and I might build the requested feature myself.
Fixed
4.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
I know this is closed, but in our project we need to hide this comment for security reasons. So here an updated patch #18 for 4.2.2. For in case it is useful for somebody