ag-admin/OpenSlides
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Include current configs in old history snapshots

Browse Source
This commit is contained in:
FinnStutzenstein 2019-08-01 11:47:09 +02:00
parent 0383f824d7
commit 4d4b9ac997
2 changed files with 21 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
client/src/app/core/repositories/base-has-content-object-repository.ts
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

20
openslides/core/views.py
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()