Merge pull request #4892 from FinnStutzenstein/includeConfigsInHistory

Include current configs in old history snapshots
This commit is contained in:
Finn Stutzenstein 2019-08-01 12:58:58 +02:00 committed by GitHub
commit 2f7937a27d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 21 additions and 2 deletions

View File

@ -93,7 +93,8 @@ export abstract class BaseHasContentObjectRepository<
(<any>ownViewModel)._contentObject = foreignModel; (<any>ownViewModel)._contentObject = foreignModel;
return true; return true;
} else { } else {
throw new Error(`The object is not an ${relation.VForeignVerbose}:` + foreignModel); console.warn(`The object is not an ${relation.VForeignVerbose}:` + foreignModel);
return false;
} }
// TODO: set reverse // TODO: set reverse

View File

@ -3,6 +3,7 @@ import os
from collections import defaultdict from collections import defaultdict
from typing import Any, Dict from typing import Any, Dict
from asgiref.sync import async_to_sync
from django.conf import settings from django.conf import settings
from django.contrib.staticfiles import finders from django.contrib.staticfiles import finders
from django.contrib.staticfiles.views import serve from django.contrib.staticfiles.views import serve
@ -20,6 +21,7 @@ from ..utils import views as utils_views
from ..utils.arguments import arguments from ..utils.arguments import arguments
from ..utils.auth import GROUP_ADMIN_PK, anonymous_is_enabled, has_perm, in_some_groups from ..utils.auth import GROUP_ADMIN_PK, anonymous_is_enabled, has_perm, in_some_groups
from ..utils.autoupdate import inform_changed_data from ..utils.autoupdate import inform_changed_data
from ..utils.cache import element_cache
from ..utils.plugins import ( from ..utils.plugins import (
get_plugin_description, get_plugin_description,
get_plugin_license, get_plugin_license,
@ -590,9 +592,25 @@ class HistoryDataView(utils_views.APIView):
full_data = instance.full_data.full_data full_data = instance.full_data.full_data
if full_data: if full_data:
dataset[collection][id] = full_data dataset[collection][id] = full_data
else: elif id in dataset[collection]:
del dataset[collection][id] del dataset[collection][id]
# Ensure, that newer configs than the requested timepoint are also
# included, so the client is happy and doesn't miss any config variables.
all_current_config_keys = set(config.config_variables.keys())
all_old_config_keys = set(
map(lambda config: config["key"], dataset["core/config"].values())
)
missing_keys = all_current_config_keys - all_old_config_keys
if missing_keys:
config_full_data = async_to_sync(element_cache.get_collection_full_data)(
"core/config"
)
key_to_id = config.get_key_to_id()
for key in missing_keys:
id = key_to_id[key]
dataset["core/config"][id] = config_full_data[id]
return { return {
collection: list(dataset[collection].values()) collection: list(dataset[collection].values())
for collection in dataset.keys() for collection in dataset.keys()