Comments

rpayanm’s picture

Status: Active » Needs review
StatusFileSize
new4.59 KB
rpayanm’s picture

Assigned: rpayanm » Unassigned
smussbach’s picture

Status: Needs review » Reviewed & tested by the community

I can confirm that all method calls are substituted. All tests pass, so I assume it is Reviewed & tested by the community?

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

Can the change record that announces the replacement of drupal_ucfirst be linked to this issue for traceability - once this is done it can be set back to rtbc.

smussbach’s picture

@alexpott Sorry, I do not know what you mean by that. What should how and where be linked to this issue? Do you mean #2205673?

alexpott’s picture

@smussbach we maintain change records which (amongst other things) detail when a function is being removed and what replaces it. See https://www.drupal.org/list-changes/drupal

The CR that talks about the replacement of drupal_ucfirst needs to be edited and this issue added to the list of issues it covers.

jeroent’s picture

Status: Needs work » Reviewed & tested by the community

Referenced this issue on the following change record: New Unicode component

alexpott’s picture

This issue is a prioritized change as per #2350615: [policy, no patch] What changes can be accepted during the Drupal 8 beta phase? and it's benefits outweigh any disruption. Committed 09ed945 and pushed to 8.0.x. Thanks!

  • alexpott committed 09ed945 on 8.0.x
    Issue #2361833 by rpayanm: Remove usage of drupal_ucfirst().
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.