diff --git a/lizmap_server/get_feature_info.py b/lizmap_server/get_feature_info.py index 028521b9..6b97a25d 100755 --- a/lizmap_server/get_feature_info.py +++ b/lizmap_server/get_feature_info.py @@ -80,7 +80,13 @@ def append_maptip(cls, string: str, layer_name: str, feature_id: Union[str, int] @classmethod def feature_list_to_replace( - cls, cfg: dict, project: QgsProject, relation_manager: QgsRelationManager, xml: str) -> List[Result]: + cls, + cfg: dict, + project: QgsProject, + relation_manager: QgsRelationManager, + xml: str, + bootstrap_5: bool, + ) -> List[Result]: """ Parse the XML and check for each layer according to the Lizmap CFG file. """ features = [] for layer_name, feature_id in GetFeatureInfoFilter.parse_xml(xml): @@ -120,7 +126,8 @@ def feature_list_to_replace( root = config.invisibleRootContainer() # Need to eval the html_content - html_content = Tooltip.create_popup_node_item_from_form(layer, root, 0, [], '', relation_manager) + html_content = Tooltip.create_popup_node_item_from_form( + layer, root, 0, [], '', relation_manager, bootstrap_5) html_content = Tooltip.create_popup(html_content) # Maybe we can avoid the CSS on all features ? @@ -175,9 +182,11 @@ def responseComplete(self): xml = request.body().data().decode("utf-8") + bootstrap_5 = params.get('CSS_FRAMEWORK', '').upper() == 'BOOTSTRAP5' + # noinspection PyBroadException try: - features = self.feature_list_to_replace(cfg, project, relation_manager, xml) + features = self.feature_list_to_replace(cfg, project, relation_manager, xml, bootstrap_5) except Exception as e: if to_bool(os.getenv("CI")): logger.log_exception(e) diff --git a/lizmap_server/tooltip.py b/lizmap_server/tooltip.py index 85919526..5e930af2 100755 --- a/lizmap_server/tooltip.py +++ b/lizmap_server/tooltip.py @@ -60,6 +60,7 @@ def create_popup_node_item_from_form( headers: list, html: str, relation_manager: QgsRelationManager, + bootstrap_5: bool = False, ) -> str: regex = re.compile(r"[^a-zA-Z0-9_]", re.IGNORECASE) a = '' @@ -173,9 +174,22 @@ def create_popup_node_item_from_form( if visibility and not active: active = visibility h += '\n' + SPACES - h += ( - '
  • {}
  • ' - ).format(active, regex.sub('_', node.name()), node.name()) + id_tab = regex.sub('_', node.name()) + if bootstrap_5: + h += ( + f'' + ) + else: + h += ( + f'
  • ' + f'{node.name()}' + f'
  • ' + ) headers.append(h) if lvl > 1: @@ -190,7 +204,8 @@ def create_popup_node_item_from_form( level += 1 for n in node.children(): - h = Tooltip.create_popup_node_item_from_form(layer, n, level, headers, html, relation_manager) + h = Tooltip.create_popup_node_item_from_form( + layer, n, level, headers, html, relation_manager, bootstrap_5) # If it is not root children, add html if lvl > 0: a += h diff --git a/test/test_get_feature_info.py b/test/test_get_feature_info.py index d3bc80b4..e21e32c8 100644 --- a/test/test_get_feature_info.py +++ b/test/test_get_feature_info.py @@ -239,7 +239,7 @@ def test_single_get_feature_info_ascii(client): "FILTER": "accents:\"NAME_1\" = 'Bret\\'agne'", } - rv = client.get(_build_query_string(qs, use_urllib3=True), PROJECT) + rv = client.get(_build_query_string(qs, use_urllib=True), PROJECT) data = _check_request(rv) if Qgis.QGIS_VERSION_INT <= 31700: diff --git a/test/utils.py b/test/utils.py index ec95397d..08c511d5 100644 --- a/test/utils.py +++ b/test/utils.py @@ -8,7 +8,7 @@ from PIL import Image from qgis.server import QgsBufferServerResponse -from urllib3 import request +from urllib.parse import urlencode __copyright__ = 'Copyright 2024, 3Liz' __license__ = 'GPL version 3' @@ -58,10 +58,10 @@ def xpath_text(self, path: str) -> str: return ' '.join(e.text for e in self.xpath(path)) -def _build_query_string(params: dict, use_urllib3: bool=False) -> str: +def _build_query_string(params: dict, use_urllib: bool = False) -> str: """ Build a query parameter from a dictionary. """ - if use_urllib3: - return "?" + request.urlencode(params) + if use_urllib: + return "?" + urlencode(params) query_string = '?' for k, v in params.items():