fix

Author: pcrespov

services/web/server/src/simcore_service_webserver/groups/_classifiers_service.py

@@ -122,13 +122,13 @@ async def _build_rrids_tree_view(
         service = ScicrunchResourcesService(self.app)
 
         flat_tree_view: dict[TreePath, ClassifierItem] = {}
-        for resource in await service.list_resources(include_url=True):
+        for resource in await service.list_resources():
             try:
                 validated_item = ClassifierItem(
                     classifier=resource.rrid,
                     display_name=resource.name.title(),
                     short_description=resource.description,
-                    url=resource.url,
+                    url=service.get_resolver_web_url(resource.rrid),
                 )
 
                 node = TypeAdapter(TreePath).validate_python(

services/web/server/src/simcore_service_webserver/scicrunch/_service.py

@@ -6,7 +6,7 @@
 
 from aiohttp import web
 from common_library.logging.logging_errors import create_troubleshooting_log_kwargs
-from pydantic import ValidationError
+from pydantic import HttpUrl, ValidationError
 
 from ._repository import ScicrunchResourcesRepository
 from .models import ResearchResource, ResearchResourceAtdB, ResourceHit
@@ -23,7 +23,7 @@ def __init__(self, app: web.Application):
         self._repo = ScicrunchResourcesRepository.create_from_app(app)
         self._scicrunch = SciCrunch.get_instance(self.app)
 
-    async def list_resources(self, include_url: bool = False) -> list[ResearchResource]:
+    async def list_resources(self) -> list[ResearchResource]:
         """List all research resources as domain models."""
         rows = await self._repo.list_all_resources()
         if not rows:
@@ -33,13 +33,6 @@ async def list_resources(self, include_url: bool = False) -> list[ResearchResour
         for row in rows:
             try:
                 resource_data = dict(row)
-
-                # Add resolver URL if requested
-                if include_url:
-                    resource_data["url"] = self._scicrunch.get_resolver_web_url(
-                        row.rrid
-                    )
-
                 resource = ResearchResource.model_validate(resource_data)
                 resources.append(resource)
             except ValidationError as err:
@@ -54,6 +47,10 @@ async def list_resources(self, include_url: bool = False) -> list[ResearchResour
 
         return resources
 
+    def get_resolver_web_url(self, rrid: str) -> HttpUrl:
+        """Get the resolver web URL for a given RRID."""
+        return self._scicrunch.get_resolver_web_url(rrid)
+
     async def get_resource_atdb(self, rrid: str) -> ResearchResourceAtdB | None:
         """Get resource with all database fields."""
         row = await self._repo.get_resource_by_rrid(rrid)

services/web/server/src/simcore_service_webserver/scicrunch/models.py

@@ -6,7 +6,7 @@
 import re
 from datetime import datetime
 
-from pydantic import BaseModel, ConfigDict, Field, HttpUrl, field_validator
+from pydantic import BaseModel, ConfigDict, Field, field_validator
 
 logger = logging.getLogger(__name__)
 
@@ -62,7 +62,6 @@ class ResearchResource(BaseModel):
     )
     name: str
     description: str
-    url: HttpUrl | None = None
 
     @field_validator("rrid", mode="before")
     @classmethod