MantisBT - ParaView
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0015338ParaView(No Category)public2015-02-23 15:302015-09-06 12:18
ReporterAlan Scott 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Versiongit-master 
Target Version4.4Fixed in Version4.4 
ProjectSandia
Topic Name
Typeincorrect functionality
Summary0015338: Save current settings as default (Properties) - logic error
DescriptionWhen you want to Save current settings as default (properties section), there is a very surprising and confusing logic bug. This is that you can click on the Save current settings as default before hitting the Apply, but it silently doesn't work. Please fix this.

* Linux, master, local server.
* Sources/ Sphere. Apply.
* Filters/ Calculator. Result name == X_DIFF. Formula == coordsX-Normals_X.
* Save current settings values as default (Properties section). (Note that you have not yet hit Apply.)
* Edit/ Reset Session.
* Sources/ Sphere. Apply.
* Filters/ Calculator. Note that the formula is not filled out.

Either make the Properties section of the Properties tab be saved correctly even before an apply is clicked, put up a warning message that nothing is being done, or best of all, gray out the Save current settings values as default icon when it really isn't active.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-02-23 15:30Alan ScottNew Issue
2015-02-23 15:30Alan ScottTarget Version => 4.4
2015-04-07 08:50Utkarsh AyachitAssigned To => Utkarsh Ayachit
2015-04-07 13:27Utkarsh AyachitNote Added: 0034343
2015-04-07 13:27Utkarsh AyachitStatusbacklog => gatekeeper review
2015-04-07 13:27Utkarsh AyachitResolutionopen => fixed
2015-04-07 21:36Utkarsh AyachitNote Added: 0034346
2015-04-07 21:36Utkarsh AyachitStatusgatekeeper review => customer review
2015-04-07 21:36Utkarsh AyachitFixed in Version => git-master
2015-04-29 22:24Alan ScottNote Added: 0034385
2015-04-29 22:24Alan ScottStatuscustomer review => closed
2015-09-06 12:18Utkarsh AyachitFixed in Versiongit-master => 4.4

Notes
(0034343)
Utkarsh Ayachit   
2015-04-07 13:27   
Awaiting review:

https://gitlab.kitware.com/paraview/paraview/merge_requests/44 [^]
(0034346)
Utkarsh Ayachit   
2015-04-07 21:36   
Merged in master:

https://gitlab.kitware.com/paraview/paraview/commit/1eba63cfaf99f93a58a417678c4c2eabfdf5c8ed [^]
(0034385)
Alan Scott   
2015-04-29 22:24   
Tested master, Linux, local server.