load_entity raises instead of logging when a category page has no usable schema title #202

OSW.load_entity reads the schema title outside the block that handles a failure per page. One unusable category page therefore ends the whole call, and the remaining titles are not loaded.

for category in jsondata["type"]:
    schema = (
        self.site.get_page(...).pages[0].get_slot_content("jsonschema")
    )
    schemas.append(schema)
    cls_name: str = schema["title"]          # line 1368
    if not param.model_to_use:
        if not hasattr(model, cls_name):     # line 1372

Three inputs raise here:

get_slot_content returns None for a slot the page does not have:

A page reaches this state when its type names a page that does not exist, is not a category, or has an empty jsonschema slot.

Expected

Report the fault for that page and continue with the next one, as the rest of the loop does.

• The entity construction below catches every exception, logs it with the page title, and the loop continues:
  

  osw-python/src/osw/core.py

  Lines 1407 to 1408 in d7f0116

  	except Exception as e:
  	_logger.error(f"Error creating entity from page {page.title}: {e}")

• The missing-model case four lines lower does the same: it logs with _logger.error, sets schemas_fetched = False and the loop continues with the next page:
  

  osw-python/src/osw/core.py

  Lines 1381 to 1388 in d7f0116

  	if not hasattr(model, cls_name):
  	schemas_fetched = False
  	_logger.error(
  	f"Model {cls_name} not found. Schema {category} "
  	f"needs to be fetched first."
  	)
  	if not schemas_fetched:
  	continue

Today the exception passes through the try at line 1340, whose finally only restores the cache state, and leaves load_entity. The message names neither the page nor the category, so the caller cannot tell which page is wrong.

Notes

• Present on main at d7f0116.
• fix(core): prefer a registered class over a generated one #166 changes the same block and does not change this behaviour. The case was found while testing that PR and left out of it, because it is independent of the subject of that PR.