couchgrid does not correctly retrieve record id

Bug #447512 reported by Rick Spencer on 2009-10-09
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
desktopcouch
Undecided
Rick Spencer
desktopcouch (Ubuntu)
High
Chad Miller

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

Changed in desktopcouch (Ubuntu):
importance: Undecided → High

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) on 2009-10-09
tags: added: ubuntu-karmic
Chad Miller (cmiller) on 2009-10-09
Changed in desktopcouch (Ubuntu):
status: New → Confirmed
assignee: nobody → Chad Miller (cmiller)
Changed in desktopcouch:
status: New → Confirmed
assignee: nobody → Rick Spencer (rick-rickspencer3)
Changed in desktopcouch:
status: Confirmed → Fix Committed
Changed in desktopcouch (Ubuntu):
status: Confirmed → Fix Committed
Chad Miller (cmiller) on 2009-10-12
Changed in desktopcouch:
status: Fix Committed → Fix Released
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  Edit
Everyone can see this information.

Other bug subscribers