Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WorkspaceProperty isDefault Issue #10286

Closed
OwenArnold opened this issue May 12, 2014 · 1 comment
Closed

WorkspaceProperty isDefault Issue #10286

OwenArnold opened this issue May 12, 2014 · 1 comment
Labels
Framework Issues and pull requests related to components in the Framework

Comments

@OwenArnold
Copy link
Contributor

This issue was originally TRAC 9443

If you have an algorithm with an optional WorkspaceProperty, and if you run that algorithm in child mode, then calling isDefault on your property will not behave as you expect it to. Even if you have set the property, isDefault will return true.

Looking at the implementation:

      bool isDefault() const
      {
        return  m_initialWSName == m_workspaceName;
      }

It's using the workspace names for comparision. If the workspaces are outside the ADS, then their names are not set, and so you are comparing two empty strings (returns True).

@OwenArnold OwenArnold added the Framework Issues and pull requests related to components in the Framework label Jun 3, 2015
@DanNixon DanNixon removed their assignment Jul 2, 2015
@NickDraper
Copy link
Contributor

We need to add a test of the pointer into this as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework Issues and pull requests related to components in the Framework
Projects
None yet
Development

No branches or pull requests

3 participants