On 01/20/2014 02:24 PM, Aline Manera wrote:
On 01/17/2014 04:41 PM, Rodrigo Trujillo wrote:
The xml file had wrong entries, then kimchi UI did not recognize the
tabs. This patch fixes this problem and also disables the sample plugin
by default.

Signed-off-by: Rodrigo Trujillo <rodrigo.trujillo@linux.vnet.ibm.com>
---
 plugins/sample/sample.conf           | 7 ++++++-
 plugins/sample/ui/config/tab-ext.xml | 8 ++++----
 plugins/sample/ui/tab.html           | 1 +
 3 files changed, 11 insertions(+), 5 deletions(-)
 create mode 100644 plugins/sample/ui/tab.html

diff --git a/plugins/sample/sample.conf b/plugins/sample/sample.conf
index c4e80f7..1a8d6b4 100644
--- a/plugins/sample/sample.conf
+++ b/plugins/sample/sample.conf
@@ -1,7 +1,12 @@
 [kimchi]
-enable = True
+enable = False
 plugin_class = "Drawings"
 uri = "/plugins/sample"

 [/]
 tools.trailing_slash.on = False
+
+
+[/ui/tab.html]
+tools.staticfile.on: True
+tools.staticfile.filename:  kimchi.config.get_plugins_dir() + '/sample/ui/tab.html'

I was expecting have a plugins/sample/sample.conf.in with:

tools.staticfile.on: True
tools.staticfile.filename: '@pluginddir@/sample/ui/tab.html'

Then we add it to Makefile.am to do the replacement during build process.

So when we get kimchi installed the plugins/sample/sample.conf file will have the correct full path

Does that make sense for you?

Hi Aline, I considered and thought about your approach, but I did not implemented it because it will only work for this plugin, which is built together with whole Kimchi code.
I my opinion, the sample plugin should be a guide for how to develop new plugins (for now, at least, as we don't a documentation and the plugin framework is still being refined).
What  do you think ?


diff --git a/plugins/sample/ui/config/tab-ext.xml b/plugins/sample/ui/config/tab-ext.xml
index 948fa07..d1f8456 100644
--- a/plugins/sample/ui/config/tab-ext.xml
+++ b/plugins/sample/ui/config/tab-ext.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
-<!--<tabs-ext>
+<tabs-ext>
     <tab>
-        <title>Test</title>
-        <filePath>plugins/sample/ui/tab.html</filePath>
+        <title>Sample</title>
+        <path>plugins/sample/ui/tab.html</path>
     </tab>
-</tabs-ext>-->
\ No newline at end of file
+</tabs-ext>
diff --git a/plugins/sample/ui/tab.html b/plugins/sample/ui/tab.html
new file mode 100644
index 0000000..1562713
--- /dev/null
+++ b/plugins/sample/ui/tab.html
@@ -0,0 +1 @@
+This is a test TAB for SAMPLE plugin