Review Request 121389: Newly created project shouldn't use window of previously opened project

Jarosław Staniek staniek at kde.org
Mon Dec 8 23:49:50 GMT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121389/
-----------------------------------------------------------

(Updated Dec. 9, 2014, 12:49 a.m.)


Review request for Calligra, Andrius da Costa Ribas, Adam Pigg, Radosław Wicik, and Wojciech Kosowicz.


Changes
-------

-> 2.9


Bugs: 340768
    http://bugs.kde.org/show_bug.cgi?id=340768


Repository: calligra


Description
-------

Newly created project shouldn't use window of previously opened project

Unlike in 2.8 (https://git.reviewboard.kde.org/r/121321/) in case of server projects
we're starting a new Kexi instance and pass connection data file to its
--connection option and database name.
In case of file projects we're passing absolute file path.

As a side effect, a number of internal APIs improved:
- use const refs when reasonable
- keep relations between connection and its filename based on string key instead of pointer (so deep copies are allowed)
- removed some unused routines


Diffs
-----

  kexi/migration/importwizard.cpp 7dae2f639511d1cafb98da33cc3b5840acad3b1d 
  kexi/widget/KexiConnectionSelectorWidget.cpp aeef8ba746fa7f015c2bf18965e134d1d973c597 
  kexi/core/KexiRecentProjects.h a22a40ea071d5a78dd9aa710f3bf2d4518af0da9 
  kexi/core/KexiRecentProjects.cpp eda7070d6f40589bf5b9a7f8d52cf4f4e58082ca 
  kexi/core/kexidbconnectionset.h 686355bff2a023e6df4d222eec80ecc47329858e 
  kexi/core/kexidbconnectionset.cpp a0616f0abbcca6d75e907a585cd19ead1022f1d3 
  kexi/main/KexiMainWindow.h fa46b1215676ade8fafbdc3708bcf44aedb96d68 
  kexi/main/KexiMainWindow.cpp e43d4d88857785aae7dec9a1e7be3a458d0da237 
  kexi/main/startup/KexiNewProjectAssistant.h 25b4705b303966e14c1e5db7cc3126d62cf2f5bf 
  kexi/main/startup/KexiNewProjectAssistant.cpp 78a3e08c6248a18a2c1d150676fe0bcf1118846e 

Diff: https://git.reviewboard.kde.org/r/121389/diff/


Testing
-------

Works OK for any combination of file/server projects


Thanks,

Jarosław Staniek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20141208/979017d4/attachment.htm>


More information about the calligra-devel mailing list