diff --git a/core/models/survex.py b/core/models/survex.py index 1071537..b7c0b9b 100644 --- a/core/models/survex.py +++ b/core/models/survex.py @@ -277,15 +277,23 @@ class Wallet(models.Model): def get_fnames(self): '''Filenames without the suffix, i.e. without the ".jpg" ''' - dirpath = Path(settings.SCANS_ROOT, self.fpath) + dirpath = Path(settings.SCANS_ROOT, self.fpath) # does nowt as fpath is a rooted path already files = [] - if dirpath.is_dir(): + if not self.fpath: + files.append(f"Incorrect path to wallet contents: '{self.fpath}'") + return files + if not dirpath.is_dir(): + files.append(f"Incorrect path to wallet contents: '{self.fpath}'") + return files + else: try: for f in dirpath.iterdir(): if f.is_file(): - if f.name != 'contents.json' and f.name != 'walletindex.html': - files.append(Path(f.name).stem) + files.append(Path(f.name).stem) + else: + files.append(f"-{Path(f.name).stem}-") except FileNotFoundError: + files.append("FileNotFoundError") pass return files diff --git a/parsers/scans.py b/parsers/scans.py index 9cbeeab..272a78f 100644 --- a/parsers/scans.py +++ b/parsers/scans.py @@ -126,7 +126,8 @@ def load_all_scans(): print("\n ", end='') print(f"{walletname} ", end='') - wallet, created = Wallet.objects.update_or_create(walletname=walletname) + fpath = Path(settings.SCANS_ROOT, str(yeardir.stem), walletname) + wallet, created = Wallet.objects.update_or_create(walletname=walletname, fpath=fpath) wallets[walletname] = wallet # could now also load the json and use it. check &ref is correct or missing too.. if not created: diff --git a/templates/logbookentry.html b/templates/logbookentry.html index d8a4817..9090928 100644 --- a/templates/logbookentry.html +++ b/templates/logbookentry.html @@ -85,15 +85,15 @@ {% if wallets %}Wallets on this date:
{% for item in wallets %} -     {{item.walletname|safe}} - {% if item.name %} - {{item.name|safe}}
- {% else %} - {{item.get_fnames|safe}} -
- {% endif %} +     {{item.walletname|safe}} + {% if item.name %} + {{item.name|safe}}
+ {% else %} + {{item.get_fnames|safe}} +
+ {% endif %} {% empty %} - None found for this date.
+ None found for this date.
{% endfor %}
{% else %}