problems with true/false objects

[boxxxie]

here is some output from my chrome dev tools resources indexedDB menu

0
"spo::6654115829151124::api_key::aaaa"
"{"subject":"6654115829151124","predicate":"api_key","object":"aaaa"}"
1
"spo::6654115829151124::connected::"
"{"subject":"6654115829151124","predicate":"connected","object":false}"
2
"spo::6654115829151124::connected::false"
"{"subject":"6654115829151124","predicate":"connected","object":"false"}"
3
"spo::6654115829151124::connected::true"
"{"subject":"6654115829151124","predicate":"connected","object":true}"

there is a quirk near the top where the value of false is not being represented in the index key of "spo::6654115829151124::connected::"

[mcollina]

The issue is on the utilities/genKey function, where each value is checked
to be truthy. Instead, we should check if a value is not null or undefined.

That is the most used function in LevelGraph, and it has been heavily
optimized. I'll try to fix this soon without causing a perf regression.

Eventually, this will be released as 0.8.0, as current master has some new
optimizations!

[mcollina]

Can you please verify if this is causing any other problems apart from key discrepancies?

[mcollina]

This should be fixed in the latest master. Can you please give it a spin and in case reopen?

[boxxxie]

do i git clone from master or can i use npm? also do i have to build (i'm only using levelgraph with indexedDB right now).

[mcollina]

Clone, npm install and then run ./browserify.sh and grab the latest build.

Let me know if you have problems, I can upload one for you.

[boxxxie]

seems to be working. thanks a lot.

[boxxxie]

actually, i spoke too soon. it seems that with the current build the object part still doesn't get indexed correctly (left blank for false values).

so, this is still an open issue.

however my program is now working properly (it isn't relying on the index for the object).

[mcollina]

Have you verified with a clean database? It does not fix previous values,
but it works properly here (I added a unit test). In case it is not fixed
yet, can you please provide some code to reproduce?

[mcollina]

Released as v0.8.0.