Review Request: Determine template type by reading a desktop file

[Miha Čančula]

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

(Updated Nov. 16, 2012, 3:16 p.m.)


Review request for KDevelop and Milian Wolff.


Changes
-------

Remove code duplication when creating a context menu action

Add a limit to recursive search of template description files


Description (updated)
-------

Determine template type by reading a desktop file

Recursively scans the file's directory and its parents for a template description file. The determined template type is used to add a context menu action, as well as for the template preview.


Diffs (updated)
-----

  plugins/filetemplates/filetemplatesplugin.h 18c9b20a02579ea7e8312f10619ba5e28ff04886 
  plugins/filetemplates/filetemplatesplugin.cpp 8912c120a32387647c107031d713549989bfe5b2 
  plugins/filetemplates/templatepreview.h 77e049f1300c0768f5d436635e356fa7d63e8f48 
  plugins/filetemplates/templatepreview.cpp f28089e0ab5f8e7ae6c3dc48483eea7df3b0f34b 

Diff: http://git.reviewboard.kde.org/r/107304/diff/


Testing
-------

Locally, with both project templates and file templates


Screenshots
-----------

Context menu of a project template file in project manager
  http://git.reviewboard.kde.org/r/107304/s/823/
Context menu when editing a class template
  http://git.reviewboard.kde.org/r/107304/s/824/


Thanks,

Miha Čančula

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20121116/eadd9c6d/attachment.html>