Mod+Shift+<number> fails to retag with python wmiirc

[GoogleCodeExporter]

What steps will reproduce the problem?

1. run wmii with python wmiirc
2. Retag client with Mod+Shift+[0-9]

What is the expected result? What do you see instead?

Traceback (most recent call last):
  File "/home/robert/.wmii-hg/pygmi/event.py", line 102, in dispatch
    self.events[event](args)
  File "/home/robert/.wmii-hg/pygmi/event.py", line 267, in dispatch
    return self._keys[key](key)
  File "/home/robert/.wmii-hg/wmiirc.py", line 260, in <lambda>
    lambda k: setattr(Client('sel'), 'tags', i)),
  File "/home/robert/.wmii-hg/pygmi/fs.py", line 172, in __set__
    dir[self.key] = val
  File "/home/robert/.wmii-hg/pygmi/fs.py", line 87, in __setitem__
    self.ctl(key, val)
  File "/home/robert/.wmii-hg/pygmi/fs.py", line 68, in ctl
    client.awrite(self.ctl_path, ' '.join(args))
TypeError: sequence item 1: expected string, int found

What version of the product are you using (wmii -v)? On what operating
system (uname -a)?

wmii-hg2725+, ©2010 Kris Maglione
Linux 2.6.32-5-686 #1 SMP Tue Jun 1 04:59:47 UTC 2010 i686 GNU/Linux

Please provide any additional information below.

Reverting one line introduced in revision 2724 solves the problem for me:

diff --git a/alternative_wmiircs/python/pygmi/fs.py b/alternative_wmiircs
--- a/alternative_wmiircs/python/pygmi/fs.py
+++ b/alternative_wmiircs/python/pygmi/fs.py
@@ -248,10 +248,10 @@
     fullscreen = Dir.toggle_property('fullscreen')
     group  = Dir.ctl_property('group')
     pid    = Dir.ctl_property('pid')
-    tags   = Dir.ctl_property('tags')
     urgent = Dir.toggle_property('urgent')

     label = Dir.file_property('label', writable=True)
+    tags  = Dir.file_property('tags', writable=True)
     props = Dir.file_property('props')

     def kill(self):
@@ -444,7 +444,7 @@

     def swap(self, src, dest):
         self.send(src, dest, cmd='swap')
-    
+
     def nudge(self, frame, dir, amount=None):
         frame = self.framespec(frame)
         self['nudge'] = '%s %s %s' % (frame, dir, str(amount or ''))

Original issue reported on code.google.com by terror.m...@googlemail.com on 15 Jun 2010 at 4:15

Attachments:

• wmii2724fix.diff

[GoogleCodeExporter]

This issue was closed by revision 5a3018702f.

Original comment by maglion...@gmail.com on 15 Jun 2010 at 4:21

• Changed state: Fixed