Merge lp://staging/~mefrio-g/light-software-center/fix-search into lp://staging/light-software-center

Proposed by Mario Guerriero
Status: Merged
Merged at revision: 43
Proposed branch: lp://staging/~mefrio-g/light-software-center/fix-search
Merge into: lp://staging/light-software-center
Diff against target: 144 lines (+55/-5)
4 files modified
src/Backend/AppsManager.vala (+38/-4)
src/Backend/Backend.vala (+2/-1)
src/Frontend.vala (+3/-0)
src/Widgets/Pages/AppsView.vala (+12/-0)
To merge this branch: bzr merge lp://staging/~mefrio-g/light-software-center/fix-search
Reviewer Review Type Date Requested Status
Lubuntu Software Center Team Pending
Review via email: mp+114592@code.staging.launchpad.net

Description of the change

* Implemented a search function (not yet tested but it should work)
* Made some function private
* Used stderr to print some error messages but we need to decide if we want to use sdtout/sdterr way or the GLib.erro/warning one

To post a comment you must log in.
Revision history for this message
Stephen Smally (stephen-smally) wrote :

Wouldn't be better to implement the search via sqlite?

Revision history for this message
Michael Rawson (michael.rawson) wrote :

> Wouldn't be better to implement the search via sqlite?

You mean sqlheavy?

Revision history for this message
Stephen Smally (stephen-smally) wrote :

Just the same, SQLHeavy is a wrapper for Sqlite3

Revision history for this message
Mario Guerriero (mefrio-g) wrote :

Yes I'll implement it using database, it's better

Revision history for this message
Stephen Smally (stephen-smally) wrote :

Thanks, or we will end having ghost applications.

43. By Mario Guerriero

fixed searching using sqlheavy

Revision history for this message
Mario Guerriero (mefrio-g) wrote :

I implemented the searching using SQLheavy in rev 43

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches