Variables deletion reloaded

[hgohil2805]

• Works for various levels of dependencies between variables.
• removed the trash all method as it's not being used anywhere

Fixes CDAT/cdat#897

[hgohil2805]

@remram44 Can you please make sure everything is OK?

@cameronsmith1 , @doutriaux1 can you please try this branch?

[remram44]

What is this second loop for?

[hgohil2805]

Second loop is to generate the popup errors of dependency if there are any
in the final list..

On Friday, March 6, 2015, Remi Rampin notifications@github.com wrote:

In vistrails/gui/uvcdat/definedVariableWidget.py
#22 (comment):

     for v in self.getSelectedDefinedVariables():

•        self.deleteVariable(v.id)
  

•        self.deleteVariable(v.id,count)
  

What is this second loop for?

—
Reply to this email directly or view it on GitHub
https://github.com/UV-CDAT/VisTrails/pull/22/files#r25926962.

[remram44]

Can you replace "count" "1" and "2" with a readable parameter?

[hgohil2805]

OK I will.

On Friday, March 6, 2015, Remi Rampin notifications@github.com wrote:

In vistrails/gui/uvcdat/definedVariableWidget.py
#22 (comment):

     for v in self.getSelectedDefinedVariables():

•        self.deleteVariable(v.id)
  

•        self.deleteVariable(v.id,count)
  

Can you replace "count" "1" and "2" with a readable parameter?

—
Reply to this email directly or view it on GitHub
https://github.com/UV-CDAT/VisTrails/pull/22/files#r25954614.

[remram44]

I had to rewrite the whole thing but here we go.