Hawkin
/
arangodb
mirror of https://gitee.com/bigwinds/arangodb
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

do not ignore document revision (#6851)

This commit is contained in:
Simon 2018-10-12 11:05:04 +02:00 committed by Jan
parent cbd44e0e2e
commit 680dbccd58
1 changed files with 15 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
arangod/RestHandler/RestDocumentHandler.cpp
View File

@ -212,8 +212,7 @@ bool RestDocumentHandler::readSingleDocument(bool generateBody) {
TRI_voc_rid_t ifRid = extractRevision("if-match", isValidRevision);
if (!isValidRevision) {
ifRid =
UINT64_MAX; // an impossible rev, so precondition failed will happen
ifRid = UINT64_MAX; // an impossible rev, so precondition failed will happen
}
VPackBuilder builder;
@ -382,14 +381,18 @@ bool RestDocumentHandler::modifyDocument(bool isPatch) {
// extract the revision, if single document variant and header given:
std::shared_ptr<VPackBuilder> builder;
if (!isArrayCase) {
TRI_voc_rid_t revision = 0;
bool isValidRevision;
revision = extractRevision("if-match", isValidRevision);
TRI_voc_rid_t headerRev = extractRevision("if-match", isValidRevision);
if (!isValidRevision) {
revision = UINT64_MAX; // an impossible revision, so precondition failed
headerRev = UINT64_MAX; // an impossible revision, so precondition failed
}
if (headerRev != 0) {
opOptions.ignoreRevs = false;
}
VPackSlice keyInBody = body.get(StaticStrings::KeyString);
if ((revision != 0 && TRI_ExtractRevisionId(body) != revision) ||
TRI_voc_rid_t revInBody = TRI_ExtractRevisionId(body);
if ((headerRev != 0 && revInBody != headerRev) ||
keyInBody.isNone() || keyInBody.isNull() ||
(keyInBody.isString() && keyInBody.copyString() != key)) {
// We need to rewrite the document with the given revision and key:
@ -398,18 +401,19 @@ bool RestDocumentHandler::modifyDocument(bool isPatch) {
VPackObjectBuilder guard(builder.get());
TRI_SanitizeObject(body, *builder);
builder->add(StaticStrings::KeyString, VPackValue(key));
if (revision != 0) {
if (headerRev != 0) {
builder->add(StaticStrings::RevString,
VPackValue(TRI_RidToString(revision)));
VPackValue(TRI_RidToString(headerRev)));
} else if (!opOptions.ignoreRevs && revInBody != 0) {
builder->add(StaticStrings::RevString,
VPackValue(TRI_RidToString(headerRev)));
}
}
body = builder->slice();
}
if (revision != 0) {
opOptions.ignoreRevs = false;
}
}
// find and load collection given by name or identifier