MantisBT - ParaView
View Issue Details
0009866ParaViewBugpublic2009-11-09 10:412010-02-22 20:29
Ken Moreland 
Utkarsh Ayachit 
highminorhave not tried
closedfixed 
 
3.83.8 
0009866: User's copy of old server connection definitions override new system-wide definitions
Consider an organization that provides one or more parallel visualization cluster resources and also distributes ParaView on the systems for users to take advantage of them. In the system setup of ParaView, there will be a site-specific default_servers.pvsc file defining how the connections are made.

When a user first starts ParaView, these connection definitions are copied into the user's local servers.pvsc file. The intention is to save the server launching arguments the user last used as the default for the next invocation (see closed bug 0005487).

The problem is that the system administrators will inevitably have to make changes to how server connections are made, which will be done to the system-wide default_servers.pvsc. However, because ParaView has already copied the old definitions to the user's personal server connection settings, the old entries override the new entries.

The solution to to change the behavior of ParaView to NOT copy the system server connection settings to the user's personal server connections file. Instead, the values to use for the server connections should be stored in the user's .ini file, separate from the server connection definitions. So when ParaView goes to connect to a server, it first reads the connection definitions from the appropriate pvsc files. It then gets the default arguments from the user's .ini settings file and, if they exist, overwrite the defaults in the pvsc file. On a successful connection, new default settings are saved back to the .ini settings file. No changes to any .pvsc file happens unless the user has "added" or "edited" one of the connections (and the user should probably not be able to edit one of the system connections, which I think was the behavior a while ago).
No tags attached.
Issue History
2009-11-09 10:41Ken MorelandNew Issue
2009-12-11 16:15Alan ScottNote Added: 0018827
2009-12-11 16:15Alan ScottPrioritynormal => high
2009-12-11 16:15Alan ScottTarget Version => 3.8
2010-01-11 10:42Utkarsh AyachitStatusbacklog => tabled
2010-01-11 10:42Utkarsh AyachitAssigned To => Utkarsh Ayachit
2010-01-11 13:52Utkarsh AyachitNote Added: 0019086
2010-01-11 13:52Utkarsh AyachitStatustabled => @80@
2010-01-11 13:52Utkarsh AyachitFixed in Version => 3.8
2010-01-11 13:52Utkarsh AyachitResolutionopen => fixed
2010-02-22 20:29Alan ScottNote Added: 0019591
2010-02-22 20:29Alan ScottStatus@80@ => closed

Notes
(0018827)
Alan Scott   
2009-12-11 16:15   
I am bumping this up to high priority, since it impacts ARL. Should be fixed for 3.8. If it is hard or time consuming, lets talk.
(0019086)
Utkarsh Ayachit   
2010-01-11 13:52   
pqServerStartups was saving user selected values for
global servers in the local servers file. It was meant to save only the user's
choices but it ended up saving all the xml defined for the global server. This
was unncessary in the first place, since pqSimpleServerStartup already has code
to save the user's choice for options in the application settings i.e.
registry. Hence removing this code that updates the user's servers.pvsc file.
However note that now, the "save" attribute on the <Option /> element becomes
critical if you want the user's selection to be preserved across sessions. Look
at http://www.paraview.org/Wiki/ParaView:Server_Configuration [^] for details on the
"save" attribute.

/cvsroot/ParaView3/ParaView3/Qt/Core/pqServerStartups.cxx,v <-- Qt/Core/pqServerStartups.cxx
new revision: 1.20; previous revision: 1.19
(0019591)
Alan Scott   
2010-02-22 20:29   
Tested client/ remote server. Random number was random. Tested Linux, trunk.

Also, feedback from reporter of this bug is that this fix is good.