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.
Just come across what looks like a bug to me but asking the question instead of submitting a patch.
line 1228 of oa_core/includes/oa_core.util.inc within oa_core_entity_build_display
this line
$display['title'] = t('A @type', array('@type' => array('@type' => node_type_get_name($entity->type))));
I am getting into it and it only returns me A looks like it's been wrapped in an extra array. Should it not be
$display['title'] = t('A @type', array('@type' => node_type_get_name($entity->type)));
That way the output is then eg A Group
Comment | File | Size | Author |
---|---|---|---|
#4 | oa_core-fix-t-function-usage-bug-2782963-4-7-x-2-66.patch | 630 bytes | lionslair |
Comments
Comment #2
mpotter CreditAttribution: mpotter at Phase2 commentedThat definitely looks like a bug. Go ahead and post a patch so you'll get credit for it and I'll commit it.
That code is only called when listing entities that the user doesn't have permission to view so it's probably never gotten tested.
Comment #3
lionslair CreditAttribution: lionslair as a volunteer commentedWill try to do on the weekend else will do on Monday.
In my use case that is pretty much the situation which is how I found it. But wanted to ask before calling it a bug.
Will reply to this with patch in next few days.
Comment #4
lionslair CreditAttribution: lionslair as a volunteer commentedAttached is the patch file. This should work.
Comment #5
lionslair CreditAttribution: lionslair as a volunteer commentedComment #6
lionslair CreditAttribution: lionslair as a volunteer commentedTwo releases have now been made since I submitted this. Is there an issue with the patch provided?
Comment #7
mpotter CreditAttribution: mpotter at Phase2 commentedSorry for the delay on this. Committed to c9d16e1 in oa_core. The last two releases were security-related releases.