See ucfirst - it [ucfirst()] returns the changed string and you're ignoring it - hence the warning;