SCM Client Api - Revision not assigned in FileObjectWrapper

Issue #846 resolved
Boris van Katwijk created an issue

The constructor FileObjectWrapper takes a revision String as argument but does not assign it to the revision field.

This causes incorrect behaviour when using its methods getContent(), getChildren(), and others.

In our project we have fixed this simply by assigning the parameter to the field.

    public FileObjectWrapper(ClientRepositoryBrowser repositoryBrowser,
        String revision, FileObject file)
    {
        this.repositoryBrowser = repositoryBrowser;
        this.file = file;

        //Fix:
        this.revision = revision;
    }

Noticed in 1.34, but as far as I can see this is still present in the latest of the 1.x branch.

Comments (2)

  1. Log in to comment