[NCBC-79] Store mode example using replace is incorrect Created: 26/Jun/12 Updated: 18/Jan/13 Resolved: 18/Jan/13 |
|
| Status: | Closed |
| Project: | Couchbase .NET client library |
| Component/s: | docs |
| Affects Version/s: | 1.1.6 |
| Fix Version/s: | 1.2.1 |
| Type: | Bug | Priority: | Major |
| Reporter: | Perry Krug | Assignee: | John Zablocki |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
This page: http://www.couchbase.com/docs/couchbase-sdk-net-1.1/couchbase-sdk-net-retrieve-set.html#table-couchbase-sdk_net_executestore-expiresat
A couple things wrong: -First, the example uses "Storemode.replace", yet the description says: Store a value using the specified key, whether the key already exists or not. Will overwrite a value if the given key/value already exists. (that's not what replace does) -Second, according to the layout of the docs, "replace" should be an update operation, not a store operation. However, it is a "storemode" so there may need to be some cross referencing? -Third, the parent page (http://www.couchbase.com/docs/couchbase-sdk-net-1.1/api-reference-set.html) lists the ExecuteStore methods out of order compared to how they are shown on the child page |
| Comments |
| Comment by John Zablocki [ 10/Jan/13 ] |
| Descriptions updated and pull request was sent. Not sure about the ordering since that's part of the merge process. I've ordered them correctly as far as I can tell, but will double check after merge and assign to docs team if that's not the case. |
| Comment by John Zablocki [ 18/Jan/13 ] |
| Docs merged and deployed. |