Can't change folder settings in Save Action (Slovak)

Bug #232436 reported by Gabriel Čenkei
4
Affects Status Importance Assigned to Milestone
Phatch
Fix Released
Critical
Stani
phatch (Ubuntu)
Fix Released
Medium
Stani

Bug Description

Distro: Ubuntu 8.04
Language: Slovak
Phatch: 0.1.3 default install from repo

Bug: After added a "Save Action" and try to change the "In" folder setting the input field got freezed.
Only restart of application can revert to the default state. The bug is executed right after clicking on the folder settings filed.
Any modification of the default folder setting is impossible.

Probably some unicode related thing only in international language versions with some unusual characters.
See attached screens to this report.

I am translating the Slovak version. If the bug can not be easily solved, temporally I can change the affected
translation string into ASCII chars. Don't know if it's a godd idea.

BTW: Thank You for this nice piece of app. It definitly finded its way on my desktop.

Revision history for this message
Gabriel Čenkei (gabox) wrote :
Revision history for this message
Stani (stani) wrote :

Thanks for reporting this critical bug, as it makes Phatch unusable on Slovak. Right now the easy workaround is to use Phatch in English. You can do this by editing your menus and give this command line to start Phatch:
phatch -l en

Another question about the translations: I think there shouldn't be a ":" after the translated word, so this:
<zlozka:> should be <zlozka> (of course with the accents but I don't know how to type them).

Changed in phatch:
assignee: nobody → stani
importance: Undecided → Critical
status: New → Confirmed
Revision history for this message
Stani (stani) wrote :

Marking this as high for the phatch package in Ubuntu, as it makes Phatch unusable on Slovak.

Changed in phatch:
assignee: nobody → stani
status: New → Confirmed
Revision history for this message
Scott Kitterman (kitterman) wrote :

Subscribing motu-sru to get approval for upload to hardy-proposed.

Changed in phatch:
importance: Undecided → Medium
Revision history for this message
Stani (stani) wrote :

I've commited a fix in bazaar. If you are familiar with bazaar please try it out. I will upload the fix later to my ppa and I want to fix another bug before doing a backport for Ubuntu.

$ bzr diff
=== modified file 'phatch/pyWx/lib/popup.py'
--- phatch/pyWx/lib/popup.py 2008-03-05 11:40:23 +0000
+++ phatch/pyWx/lib/popup.py 2008-05-23 14:47:56 +0000
@@ -81,9 +81,12 @@
             %(self.instance.__class__.__name__,self.method)

 #---base controls
+def untranslated(self,x):
+ return x
+
 class _Ctrl(object):
- _to_local = str
- _to_english = str
+ _to_local = untranslated
+ _to_english = untranslated

     def Set(self,value):
         self.SetValue(self._to_local(value))

$ bzr commit -m "fix unicode variables"
Committing to: /home/stani/sync/python/phatch/trunk/
modified phatch/pyWx/lib/popup.py
Committed revision 527.

Changed in phatch:
status: Confirmed → In Progress
status: Confirmed → Fix Committed
Revision history for this message
Lufa (yuriy.smetana) wrote :

The same problem with Ukrainian localisation.
Thank you.

Revision history for this message
Stani (stani) wrote :

Can you please confirm the latest release fixes your problem?
http://sd-2986.dedibox.fr/photobatch/download/index.html

Or you can also try it from my PPA:
https://launchpad.net/~stani/+archive

Please confirm asap so I can upload this to Debian and Ubuntu. I have verified that it worked myself by launching Phatch as "phatch -l sk".

Changed in phatch:
status: Fix Committed → Fix Released
Revision history for this message
Gabriel Čenkei (gabox) wrote :

I can confirm it works now. Good work. Thank you.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package phatch - 0.1.5-1

---------------
phatch (0.1.5-1) unstable; urgency=medium

  [ Stani M ]
  * New upstream release(Closes LP: #228259, #232436, #236282, #236462, #237359)
  * debian/control:
    - Added pkg-config and python-nautilus to build-depends, as
      setup.py uses nautilus-python.pc to look for the nautilus
      extensions path. Closes: #475233.
  * debian/patches:
    - All patches of Ubuntu and Debian are implemented upstream and are removed
      from the package

  [ Piotr Ożarowski ]
  * Bump Standards-Version to 3.8.0 (no changes needed)
  * debian/copyright wrapped at column 80 (where possible)

phatch (0.1.3-2) unstable; urgency=low

  * debian/patches/from_upstream_fix_themes.patch:
    - Fix Phatch not working properly with some themes.
      Thanks to Stani for the patch.

 -- Cesare Tirabassi <email address hidden> Wed, 30 Jul 2008 21:09:03 +0100

Changed in phatch:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.