Found and fixed. Thanks.
-Damien
On Aug 28, 2008, at 7:03 PM, Michael Hendricks wrote:
On Thu, Aug 28, 2008 at 09:28:06PM -0000, [EMAIL PROTECTED] wrote:
Author: damien
Date: Thu Aug 28 14:28:06 2008
New Revision: 689996
URL: http://svn.apache.org/viewvc?rev=689996&view=rev
Log:
Fix for missing document ID when creating new documents view bulk
post, and fix the welcome message.
Thanks Damien. I found another bug on the json_term_changes branch.
When I examine a view and CouchDB tries to spawn a new javascript
instance, I got a 500 error and the following output:
{"error":"error",
"reason":"{{badarg,[
{erlang,binary_to_list,[\"javascript\"]},
{couch_query_servers,handle_call,3},
{gen_server,handle_msg,5},
{proc_lib,init_p,5}]},
{gen_server,call, [couch_query_servers,{get_port,\"javascript
\"}]}}"}
The patch below fixes it. I'm not sure why the test suite didn't
catch
it though. The only other problem I've found on this branch
(creating a
database that already exists) is a regression on trunk too. I sent a
patch for that on 23 August.
--- couch_query_servers.erl (revision 690030)
+++ couch_query_servers.erl (working copy)
@@ -210,7 +210,7 @@
end,
{reply, Result, {QueryServerList, LangPorts -- [LangPort]}};
false ->
- case lists:keysearch(binary_to_list(Lang), 1,
QueryServerList) of
+ case lists:keysearch(Lang, 1, QueryServerList) of
{value, {_, ServerCmd}} ->
{reply, {empty, ServerCmd}, {QueryServerList, LangPorts}};
false -> % not a supported language
--
Michael
