View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0014760 | ParaView | (No Category) | public | 2014-05-28 14:02 | 2014-08-29 11:00 | ||||
Reporter | Utkarsh Ayachit | ||||||||
Assigned To | Cory Quammen | ||||||||
Priority | normal | Severity | minor | Reproducibility | have not tried | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | git-master | ||||||||
Target Version | 4.2 | Fixed in Version | 4.2 | ||||||
Summary | 0014760: developer tag: Application often fail to save settings file. | ||||||||
Description | In a custom application I am getting "Saving settings file failed" errors on quit. The problem is because the "/home/utkarsh/.config/temtomography/" directory doesn't exist and hence saving of "UserSettings" fails. We should fix as follows: + Don't bother saving settings file is the settings weren't changed. This will also avoid saving settings file unnecessarily -- less IO, the better. + When saving settings file, we need to ensure the directories exists and are created if not. Also remove the need for applications to call vtkInitializationHelper::SetApplicationName. pqApplicationCore in its contructor can pass the application name set on the QCoreApplication. This avoids the need to update existing Qt applications. Also remove the SetApplicationName from branded_paraview_initializer.cxx.in. | ||||||||
Tags | No tags attached. | ||||||||
Project | Sandia | ||||||||
Topic Name | 14760_fix_settings_saving | ||||||||
Type | incorrect functionality | ||||||||
Attached Files | |||||||||
Relationships | |
Relationships |
Notes | |
(0032699) Cory Quammen (developer) 2014-05-30 16:24 |
Topic posted to gerrit for review because the addition of the organization to the naming scheme might be controversial: http://review.source.kitware.com/#/t/4253/ [^] |
(0032806) Cory Quammen (developer) 2014-06-09 13:21 |
I have a fix coming for this based on another topic merged into next. I will merge this in when that one is merged into master. |
(0032819) Cory Quammen (developer) 2014-06-10 09:01 |
commit 79b76262c291e03ff29fe80018170b18e00f0361 Author: Cory Quammen <cory.quammen@kitware.com> Date: Tue Jun 10 08:40:14 2014 -0400 Fix pqProxyWidgetDialog to activate OK button when enter is pressed Change-Id: I3766b8805b4e71985b4faf2305815f0067ecf2ac |
(0032851) Utkarsh Ayachit (administrator) 2014-06-12 08:59 |
SUMMARY --------------------------------------------- Topics merged into master: (VTK) 14244_fix_stereo_image_captures 14696_min_max_labels 14760_fix_settings_saving 14792_fix_remote_volume_rendering fix_some_warnings static-plugins-in-servers |
(0032901) Alan Scott (manager) 2014-06-18 22:16 |
Developer tag in git. Closing untested. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2014-05-28 14:02 | Utkarsh Ayachit | New Issue | |
2014-05-28 14:02 | Utkarsh Ayachit | Assigned To | => Cory Quammen |
2014-05-30 16:24 | Cory Quammen | Note Added: 0032699 | |
2014-06-09 13:21 | Cory Quammen | Note Added: 0032806 | |
2014-06-09 13:21 | Cory Quammen | Status | backlog => active development |
2014-06-10 09:01 | Cory Quammen | Topic Name | => 14760_fix_settings_saving |
2014-06-10 09:01 | Cory Quammen | Note Added: 0032819 | |
2014-06-10 09:01 | Cory Quammen | Status | active development => gatekeeper review |
2014-06-10 09:01 | Cory Quammen | Fixed in Version | => git-next |
2014-06-10 09:01 | Cory Quammen | Resolution | open => fixed |
2014-06-12 08:59 | Utkarsh Ayachit | Fixed in Version | git-next => git-master |
2014-06-12 08:59 | Utkarsh Ayachit | Status | gatekeeper review => customer review |
2014-06-12 08:59 | Utkarsh Ayachit | Note Added: 0032851 | |
2014-06-17 09:21 | Utkarsh Ayachit | Target Version | => 4.2 |
2014-06-18 22:16 | Alan Scott | Note Added: 0032901 | |
2014-06-18 22:16 | Alan Scott | Status | customer review => closed |
2014-08-29 11:00 | Utkarsh Ayachit | Fixed in Version | git-master => 4.2 |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |