> On Feb. 18, 2016, 11:35 a.m., Aleix Pol Gonzalez wrote: > > src/PackageModel/PackageWidget.cpp, line 279 > > <https://git.reviewboard.kde.org/r/127107/diff/1/?file=444572#file444572line279> > > > > Why is `startSearch()` needed now?
m_proxyModel->reset() removes the packages from the model, but it also invalidates the filter. Since the search box keeps its value a call to startSearch() restores the same filter. Section filters (package status, category, ...) are already preserved. - Carlo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127107/#review92526 ----------------------------------------------------------- On Feb. 19, 2016, 12:31 a.m., Carlo Vanini wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127107/ > ----------------------------------------------------------- > > (Updated Feb. 19, 2016, 12:31 a.m.) > > > Review request for Kubuntu and Muon Package Management Suite. > > > Bugs: 332530 > http://bugs.kde.org/show_bug.cgi?id=332530 > > > Repository: muon > > > Description > ------- > > When the cache is reloaded libqapt deletes Package objects and creates them > anew. This results in dangling pointers in muon models. > > When "Preview Changes" is shown we have two models with lists of packages, > and both should be cleared and repopulated. > > The cache reload can be triggered by pinning/unpinning (lock/unlock) a > package. > > > Diffs > ----- > > src/MainWindow.cpp 466ea4bc00773dbd5353a477a10bbb125d021287 > src/PackageModel/PackageWidget.h 681b59e6887b04140f899f214d0b8ddc1fc2de3c > src/PackageModel/PackageWidget.cpp 264db785de78295d1e688da77d850993abe7c6d6 > src/StatusWidget.cpp ecd17ea402fd93b00ab6fa0773464bbc8a600990 > > Diff: https://git.reviewboard.kde.org/r/127107/diff/ > > > Testing > ------- > > * mark a package for install/remove > * click on "Preview Changes" > * right-click on a package and select "Lock at Current Version" > > Without patch: clicking on "Back" crashes muon > With patch: no crash, but as a side effect all changes to packages are lost > when the cache is reloaded. > > > Thanks, > > Carlo Vanini > >
-- kubuntu-devel mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/kubuntu-devel
