Ubuntu
desktopcouch package

couchgrid does not correctly retrieve record id

Bug #447512 reported by Rick Spencer
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
desktopcouch
Fix Released
Undecided
Rick Spencer
desktopcouch (Ubuntu)
Fix Released
High
Chad Miller
Ubuntu ubuntu-9.10

Bug Description

Binary package hint: desktopcouch

the couchgrid selected_record_ids property fails because when the treeview is populated it stores the record's "key" property, which used to work. But the api was changed so that the records value["_id"] was required instead.

This can be fixed in the CouchGrid module by doing this change:

@@ -212,7 +213,7 @@
                     pass

             #set the last value as the document_id, and append
- row[-1] = r.key
+ row[-1] = r.value["_id"]
             self.list_store.append(row)

ProblemType: Bug
Architecture: i386
Date: Fri Oct 9 12:35:35 2009
DistroRelease: Ubuntu 9.10
Package: desktopcouch 0.4.2-0ubuntu1
PackageArchitecture: all
ProcEnviron:
 PATH=(custom, user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-13.42-generic
SourcePackage: desktopcouch
Uname: Linux 2.6.31-13-generic i686

Related branches

lp:~rick-rickspencer3/desktopcouch/couchgrid_id_fix
Stuart Langridge (community): Approve
Diff: 12 lines
1 file modified
desktopcouch/records/couchgrid.py (+1/-1)
lp:ubuntu/karmic/desktopcouch
Revision history for this message
Rick Spencer (rick-rickspencer3) wrote :
Changed in desktopcouch (Ubuntu):
importance: Undecided → High
Revision history for this message
Rick Spencer (rick-rickspencer3) wrote :

This is an easy fix, and necessary to make this work for Karmic.

Changed in desktopcouch (Ubuntu):
milestone: none → ubuntu-9.10
Chad Miller (cmiller)
tags: added: ubuntu-karmic
Chad Miller (cmiller)
Changed in desktopcouch (Ubuntu):
status: New → Confirmed
assignee: nobody → Chad Miller (cmiller)
Changed in desktopcouch:
status: New → Confirmed
assignee: nobody → Rick Spencer (rick-rickspencer3)
Rick Spencer (rick-rickspencer3)
Changed in desktopcouch:
status: Confirmed → Fix Committed
Changed in desktopcouch (Ubuntu):
status: Confirmed → Fix Committed
Chad Miller (cmiller)
Changed in desktopcouch:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package desktopcouch - 0.4.4-0ubuntu1

---------------
desktopcouch (0.4.4-0ubuntu1) karmic; urgency=low

  * New upstream release.
    + Include doc "txt" and translation files in sources.
    + couchgrid does not correctly retrieve record id (LP: #447512)
    + couchgrid selected_records property is buggy and should be removed for
      karmic if possible (LP: #448357)

desktopcouch (0.4.3-0ubuntu1) karmic; urgency=low

  * Include compulsory-auth INI file to be secure by default.
    (LP: #438800)
  * Make debhelper warn about files not installed to some package.
  * Shorten debhelper install paths using dh_install exlusions.
  * New upstream release:
    + couchgrid did not correctly retrieve record id (LP: #447512)
    + HTTP 401 for valid auth information when talking to couchdb over SSL
      (LP: #446516)
    + Support headless apps. (LP: #428681)
    + desktopcouch-service "ValueError: dictionary update sequence..." on
      stdout(LP: #446511)

 -- Chad MILLER <email address hidden> Mon, 12 Oct 2009 10:17:50 -0400

Changed in desktopcouch (Ubuntu):
status: Fix Committed → 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.