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.
I have tried to set different title instead of "Basics" but it still shows up as "Basics" in node page. Is this a bug?
Comment | File | Size | Author |
---|---|---|---|
#20 | node_view_only_showing_basics.patch | 1019 bytes | serenecloud |
#5 | cck_fieldgroup_tabs__node_type_in_variable_get.patch | 773 bytes | greg.harvey |
Comments
Comment #1
toadyc CreditAttribution: toadyc commentedSame thing happens to me
Comment #2
nevets CreditAttribution: nevets commentedProblem is because cck_fieldgroup_tabs_nodeapi() does not use per content type tab titles. I changed line 178 so it reads
and it seems to work correctly.
Comment #3
nicholas.alipaz CreditAttribution: nicholas.alipaz commentedDoesn't seem to fix it for me, but 178 didn't really seem like the right line from what I could tell. Could you check your alteration and let me know what the lines around that altered line read?
Comment #4
nicholas.alipaz CreditAttribution: nicholas.alipaz commentedOK, figured it out after looking a bit more...
Find:
$title = t(variable_get('cck_fieldset_tabs_residual_title', t('Basics')));
Change to:
$title = t(variable_get('cck_fieldset_tabs_residual_title_'.$node->type, variable_get('cck_fieldset_tabs_residual_title', t('Basics'))));
Thanks, I can confirm this fixes the issue for me.
Comment #5
greg.harveyPatch attached.
Comment #6
chaloalvarezj CreditAttribution: chaloalvarezj commentedJust to confirm that the patch works perfectly! Thank you!
Comment #7
jcmarco CreditAttribution: jcmarco commentedConfirmed that it works fine.
Thank you.
Comment #8
greg.harveyOk, reviewed and tested. =)
Hopefully someone will commit it soon.
Comment #9
nedjoThanks for the bug report. greg.harvey, I've assigned you CVS access, please go ahead and apply this to HEAD.
Comment #10
nedjoThanks, applied.
Comment #11
greg.harveyHi - thanks for the CVS access - unfortunately I'm on vacation so was not able to apply the patch. See you've done it now anyway. Thanks! =)
Comment #13
redraven CreditAttribution: redraven commentedIn the most recent release this appears to be only fixed for the 'edit' view of the tabs. In this case 'Basics' is replaced by the defined over-ride for the default 'Basics' name in the module config page.
When just viewing the full node, the tab still defaults to 'basic' and the change made in the module config page is not reflected in the tabs, and 'basic still appears.
Comment #14
Miria CreditAttribution: Miria commentedYes the problem still exists in the node view, which is very important because this is what displays to everyone. Anyone know how to fix this?
Comment #15
greg.harveyThere has not been an official release since this was fixed. Are you using 6.x-1.0 or 6.x-dev? This is only fixed in 6.x-dev so far. It *will* be fixed in 6.x-1.1.
If you say it's not fixed in the dev snapshot, then please re-activate the issue - otherwise leave it closed and use 6.x-dev (or wait for 6.x-1.1).
Comment #16
mrlava CreditAttribution: mrlava commentedI am using 6.x-dev and the view page tab shows 'Basics' regardless of what I set it to in admin/settings/cck-fieldgroup-tabs
Comment #17
mrlava CreditAttribution: mrlava commentedneither the dev or the recommended is displaying the correct information in the node view
Comment #18
jaymallison CreditAttribution: jaymallison commenteda quick fix for this (the title being wrong on the node view) is to add the following to the cck_fieldgroup_tabs_admin_settings() function.
This will set it site wide to something else. The code needs to be reworked a couple different ways to change it per content type. I only have one, so this worked for me.
Comment #19
greg.harvey@jaymallison #5 already contains a patch that *does* allow different titles per content type. No need to write more code.
Setting back to reviewed and tested, so the patch can be re-applied. Not sure how the changes got lost.
Comment #20
serenecloud CreditAttribution: serenecloud commentedMy patch for this issue - same as #2 - only rolled as a patch.
Comment #21
freatida CreditAttribution: freatida commentedI've applied the patch but node view is still showing Basics title.
Does this work for anyone?
Comment #22
greg.harveyYes, it works for me and several others (see comments 6, 7 & 8).
Comment #23
Sutharsan CreditAttribution: Sutharsan commentedPatch #20 works. RTBC +1
It looks like a plain bug, $node_type does not exist in cck_fieldgroup_tabs_nodeapi().
Comment #24
nedjoThanks, fixed.