Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The bartik.theme file still references html.tpl.php in the comment for
function bartik_preprocess_html(&$variables) {
Should be html.html.twig.
Comment | File | Size | Author |
---|---|---|---|
#1 | bartik-theme-html-html-twig-comment-correction-2095109-1.patch | 391 bytes | chris_hall_hu_cheng |
Comments
Comment #1
chris_hall_hu_cheng CreditAttribution: chris_hall_hu_cheng commentedCorrected comment in attached patch.
Comment #2
mradcliffePatch does what it intends to do and applies.
This issue is a follow-up to #1033116: Rename template.php to THEMENAME.theme to eliminate ambiguity around "the template file" vs. "template files" where that patch did not update all of the doc blocks.
There are 2 instances of this in the seven theme, and seven also references tpl.php in a comment within a function. Should this issue could be expanded in scope and changed to the documentation component?
Comment #3
chris_hall_hu_cheng CreditAttribution: chris_hall_hu_cheng commentedThanks for adding the tag, I will know to do that next time.
What is the fastest option? I can raise another issue and happy to submit an issue and patch for the documentation in Seven theme tomorrow.
Apologies for not looking further, I was focused on Bartik as I was sub-themeing it for my blog.
Comment #4
chris_hall_hu_cheng CreditAttribution: chris_hall_hu_cheng commentedI did it I raised and issue and patch for the references to similar problems in Seven made in #2
#2098399: Seven commenting has a number of references to .tpl.php files
Comment #5
LinL CreditAttribution: LinL commentedThis looks good, but there is already an issue to make all of these changes in one go: #2049207: [Follow up] Replace .tpl.php with .html.twig in documentation, so this one can be closed as a duplicate.
Comment #6
chris_hall_hu_cheng CreditAttribution: chris_hall_hu_cheng commentedSo I just wasted my time, also with the one for Seven, did try searching but came up with nothing (my search was obviously bad)
and the comment-er in #2 obviously wasn't aware of #2049207: [Follow up] Replace .tpl.php with .html.twig in documentation either.
Where do I start?? I don't have time to hang around in IRC for ages, ask loads of questions just to find out how to make basic contributions.
The issue you refer to is months old has over 70 comments and still isn't resolved, meanwhile allowing ignorant newcomers like me to waste time.
Contrib land is looking much more fun and agile right now? or am I missing something.
Not meaning to be negative, partly a rhetorical question but would be grateful for some pointers or compelling reason to get involve in core for Drupal 8 at all at any level if anybody has the time.
Comment #7
mradcliffeI tried searching for "tpl" in the issue queue and I didn't see it when I skimmed the list.
I have submitted duplicate issues before and it is not a problem, chris_hall_hu_cheng. I am often in the same situation - less time to hang around in IRC. If you're interested in core development in general, then the best place to get situated is probably http://drupalmentioring.org, which provides tasks that can be accomplished.
The best thing to do now is to switch over to that issue and help review the patches.
Side Note: Contrib is much easier because there is less noise.
Comment #8
chris_hall_hu_cheng CreditAttribution: chris_hall_hu_cheng commentedNo problem, I have commit rights on the contrib. blog module for D8 so have plenty to keep me busy getting that fixed up for D8 and then turning it into a proper blog module ;)