Quickly

Error in add dialog command

Bug #578710 reported by Javier Hernández
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Quickly
Fix Released
Medium
Unassigned
quickly (Ubuntu)
Fix Released
Medium
Unassigned
Lucid
Fix Released
Medium
Unassigned

Bug Description

When i add a new dialog to my application, when i run quickly design informs unable to load data/ui/Gtd3eFooDialog.ui.
required catalog unavailable: gtd3e_foo_dialog

jhernandez@jhernandez-laptop:~/bzr/gtd3e$ quickly add dialog gtd3e-foo
jhernandez@jhernandez-laptop:~/bzr/gtd3e$ rgrep foo .
./data/ui/Gtd3eFooDialog.ui: <!-- interface-requires gtd3e_foo_dialog 1.0 -->
./data/ui/Gtd3eFooDialog.ui: <object class="Gtd3eFooDialog" id="gtd3e_foo_dialog">
./data/ui/gtd3e_foo_dialog.xml:<glade-catalog name="gtd3e-foo_dialog" domain="glade-3"
./data/ui/gtd3e_foo_dialog.xml: generic-name="gtd3e-foo_dialog" parent="GtkDialog"
./gtd3e/Gtd3eFooDialog.py: new_object = builder.get_object('gtd3e_foo_dialog')
jhernandez@jhernandez-laptop:~/bzr/gtd3e$

Then, if i replace all 'gtd3e-foo' strings in './data/ui/gtd3e_foo_dialog.xml' i can run quickly design without errors.

It seems that a string.replace('-', '_') in the correct line may solve the problem.

My quickly version:
jhernandez@jhernandez-laptop:~/bzr/gtd3e$ quickly --version
Quickly 0.4.2
  Python interpreter: /usr/bin/python 2.6.5
  Python standard library: /usr/lib/python2.6

  Quickly used library: /usr/lib/python2.6/dist-packages/quickly
  Quickly data path: /usr/share/quickly
  Quickly detected template directories:
          /usr/share/quickly/templates/

Revision history for this message
Javier Hernández (javiube) wrote :

I corrected this in /usr/share/quickly/templates/ubuntu-application/add.py. When ui files are generated, i have redefined substitutions before xml has generated.

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Thanks a lot! I've slightely change the patch and pushed now in trunk :)

Should be available on next release

Changed in quickly:
status: New → Fix Committed
importance: Undecided → Medium
Changed in quickly (Ubuntu):
status: New → Triaged
importance: Undecided → Medium
David Futcher (bobbo)
tags: added: patch-forwarded-upstream
tags: added: patch-accepted-upstream
removed: patch-forwarded-upstream
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

 quickly (0.4.3-0ubuntu1) lucid-proposed; urgency=low
 .
   * New bugfix release:
     quickly core:
     - fix wrong template proposal for commands not followed by template
     ubuntu-application template:
     - change label caption from glade to design (LP: #571409)
     - fix dialog when having - in title name (LP: #578710)
     - fix unicode problems in About dialog (LP: #582584) (Petar Vasić)
     - fix the boiler plate for faulty preferences dialog code (LP: #587090)
       (Nick Veitch)
     ubuntu-application and inherited:
     - fix configure stripping team name for ppa (LP: #587314) (Frederik Elwert)
     - fix not all syntax for dependencies supported (LP: #587520) (Frederik
       Elwert)
     - fix issue and description of release: quickly release <number>
     updated translations

TestCase:
- install the new version
- check that the bugs listed in the changelog are fixed
- check that it doesn't break the normal Quicklly workflow

Changed in quickly (Ubuntu Lucid):
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :
Changed in quickly:
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted quickly into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in quickly (Ubuntu Lucid):
status: Triaged → Fix Committed
tags: added: verification-needed
Revision history for this message
Ryan Macnish (nisshh) wrote :

I have tested 0.4.3 and this bug is fixed.

Martin Pitt (pitti)
tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package quickly - 0.4.3-0ubuntu1

---------------
quickly (0.4.3-0ubuntu1) lucid-proposed; urgency=low

  * New bugfix release:
    quickly core:
    - fix wrong template proposal for commands not followed by template
    ubuntu-application template:
    - change label caption from glade to design (LP: #571409)
    - fix dialog when having - in title name (LP: #578710)
    - fix unicode problems in About dialog (LP: #582584) (Petar Vasić)
    - fix the boiler plate for faulty preferences dialog code (LP: #587090)
      (Nick Veitch)
    ubuntu-application and inherited:
    - fix configure stripping team name for ppa (LP: #587314) (Frederik Elwert)
    - fix not all syntax for dependencies supported (LP: #587520) (Frederik
      Elwert)
    - fix issue and description of release: quickly release <number>
    updated translations
 -- Didier Roche <email address hidden> Fri, 11 Jun 2010 13:18:01 +0200

Changed in quickly (Ubuntu Lucid):
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

Copied to maverick as well.

Changed in quickly (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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