[Kde-games-devel] Review Request: Port all KNewStuff2 code in KDEGames to KNewStuff3

[Parker Coates]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3343/
-----------------------------------------------------------

(Updated 2010-03-23 20:38:35.338408)


Review request for KDE Games and Frederik Gladhorn.


Changes
-------

Removed the KNS3 signal workaround as this has been fixed in trunk.


Summary
-------

I just went through KDEGames trunk and updated all uses of KNewStuff2 to KNewStuff3. Most applications use KGameThemeSelector, so only their knsrc files needed updating.

Summary of changes:

KAtomic: Updated custom level set dialog. Had to work around a bug in the KNS3 interface.

KBlocks: Uses KGameThemeSelector. Updated knsrc file.

KDiamond: Uses KGameThemeSelector. Updated knsrc file.

KGoldrunner: Has a knsrc file, but doesn't seem to use it anywhere. Updated it anyway.

Kigo: Has two knsrc files: one for saved games, one for themes. Unfortunately, both of them use the same provider URL, so the Get More Themes button in the theme selector lets you download saved games to the themes directory, which is obviously quite confusing. So I disable the button in the theme selector until that is sorted out.

KMines: Has an invalid, unused knsrc file. Updated it anyway.

KTron: Uses KGameThemeSelector. Updated knsrc file. (There doesn't seem to be anything on the server for it, though.)

KGameThemeSelector: Updated to KNewStuff3.


Diffs (updated)
-----

  trunk/KDE/kdegames/katomic/chooselevelsetdialog.h 1106088 
  trunk/KDE/kdegames/katomic/chooselevelsetdialog.cpp 1106088 
  trunk/KDE/kdegames/katomic/katomic.knsrc 1106088 
  trunk/KDE/kdegames/katomic/levelsetwidget.ui 1106088 
  trunk/KDE/kdegames/kblocks/kblocks.knsrc 1106088 
  trunk/KDE/kdegames/kdiamond/src/kdiamond.knsrc 1106088 
  trunk/KDE/kdegames/kgoldrunner/themes/kgoldrunner.knsrc 1106088 
  trunk/KDE/kdegames/kigo/src/gui/mainwindow.cpp 1106088 
  trunk/KDE/kdegames/kmines/kmines.knsrc 1106088 
  trunk/KDE/kdegames/ktron/ktron.knsrc 1106088 
  trunk/KDE/kdegames/libkdegames/CMakeLists.txt 1106088 
  trunk/KDE/kdegames/libkdegames/kgamethemeselector.cpp 1106088 

Diff: http://reviewboard.kde.org/r/3343/diff


Testing
-------

Installed and uninstalled a theme for all those games that support it.


Thanks,

Parker