Incorrect logic in ChorusHelper URI builder

[rmunn]

The ChorusHelper URI builder has the following code in GetSyncUri():

public virtual string GetSyncUri(ILfProject project)
{
    var settings = MainClass.Container.Resolve<LfMergeSettings>();
    if (!string.IsNullOrEmpty(settings.LanguageDepotRepoUri))
        return settings.LanguageDepotRepoUri;

    var uriBldr = new UriBuilder(project.LanguageDepotProjectUri) {
        UserName = Username,
        Password = Password,
        Path = HttpUtility.UrlEncode(project.LanguageDepotProject.Identifier)
    };
    return uriBldr.Uri.ToString();
}

But this logic is incorrect: if there's a LanguageDepotRepoUri set in the LfMerge global settings, it will (of course) not be project-specific and not contain a project identifier. And so if someone sets the LanguageDepotRepoUri setting in the global config file, every project will fail to Send/Receive because GetSyncUri will start returning URIs with no project identifier in them.

The logic for handling a non-empty settings.LanguageDepotRepoUri value needs to include the project identifier in the URL.