We've removed all the usages of Utility\String but there are still plenty of use statements, lets remove those now so we can start testing on PHP 7.

Also, we should do an audit of unused use statements in core, there are heaps!

Comments

xano’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +PHP 7.0 (duplicate)

The patch only removes use statements and all tests pass.

Thanks!

benjy’s picture

hussainweb’s picture

Status: Reviewed & tested by the community » Needs review
StatusFileSize
new678 bytes
new11.1 KB

There is another reference. Removing it.

benjy’s picture

Status: Needs review » Reviewed & tested by the community

Just a comment. Back to RTBC

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Part of the PHP7 critical - could be done as part of the class removal but doing it now keeps things simple. Committed ea08182 and pushed to 8.0.x. Thanks!

  • alexpott committed ea08182 on 8.0.x
    Issue #2462151 by hussainweb, benjy: Remove unused Utility\String use...

Status: Fixed » Closed (fixed)

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