[FIXED JENKINS-25559] - Proper search of the effective client name for running builds

* Fixed substitution errors in nested buildEnvVars() calls (e.g. EXECUTOR_NUMBER) (introduced by #60)
* Minor refactoring
* Moved nested call checks to <code>PerforceSCM::buildEnvVars</code>

CC: @stuartrowe