diff --git a/fdroidserver/update.py b/fdroidserver/update.py index ab94f62e..8839c531 100644 --- a/fdroidserver/update.py +++ b/fdroidserver/update.py @@ -1137,6 +1137,9 @@ def insert_localized_app_metadata(apps): if base not in apps[packageName] or not isinstance(apps[packageName][base], collections.OrderedDict): apps[packageName][base] = collections.OrderedDict() apps[packageName][base][locale] = common.file_entry(dst) + + # copy screenshots from local source code checkout into wellknown + # location in repo directory for d in dirs: if d in SCREENSHOT_DIRS: if locale == 'images': @@ -1221,8 +1224,33 @@ FASTLANE_IOS_MAP = { } +def parse_ios_screenshot_name(path): + """ + Infer type info from screenshot file name. + + Device type/name is part of the file name of iOS fastlane screenshots. + Here are some example: + * 'iPhone 8+ @ iOS 16-1.png' + * 'iPad Pro 12.9" 2gen @ iOS 16-1.png' + * '1_ipadPro129_1.1.png' + * '1_iphone6Plus_1.1.png' + """ + s = path.stem.split('@') + if len(s) >= 2: + if "iphone" in s[0].lower(): + return ("phoneScreenshots", '@'.join(s[1:])) + elif "ipad" in s[0].lower(): + return ("tenInchScreenshots", "@".join(s[1:])) + else: + return ('phoneScreenshots', s[0]) + + def insert_localized_ios_app_metadata(apps_with_packages): + if not any(pathlib.Path('repo').glob('*.ipa')): + # no IPA files present in repo, nothing to do here, exiting early + return + for package_name, app in apps_with_packages.items(): if not any(pathlib.Path('repo').glob(f'{package_name}*.ipa')): # couldn't find any IPA files for this package_name @@ -1240,6 +1268,31 @@ def insert_localized_ios_app_metadata(apps_with_packages): if key: _set_localized_text_entry(app, lang_code, key, metadata_file) + # discover available screenshots and put findings in a dict + screenshots = {} + for lang_sdir in (fastlane_dir / 'screenshots').iterdir(): + lang_code = lang_sdir.name + m = LANG_CODE.match(lang_code) + if m: + screenshots[lang_code] = {} + for screenshot in (lang_sdir).iterdir(): + if screenshot.suffix[1:] in ALLOWED_EXTENSIONS: + # asdf #TODO + device_name, screenshot_name = parse_ios_screenshot_name(screenshot) + + if not screenshots[lang_code].get(device_name): + screenshots[lang_code][device_name] = {} + screenshots[lang_code][device_name][screenshot_name] = screenshot + + # copy screenshots to repo dir + for lang_code in screenshots.keys(): + for device in screenshots[lang_code].keys(): + dest_dir = pathlib.Path('repo') / package_name / lang_code / device + dest_dir.mkdir(mode=0o755, parents=True, exist_ok=True) + for name, path in screenshots[lang_code][device].items(): + dest = dest_dir / (name.replace(" ", "_").replace("\t", "_") + path.suffix) + _strip_and_copy_image(str(path), str(dest)) + def scan_repo_files(apkcache, repodir, knownapks, use_date_from_file=False): """Scan a repo for all files with an extension except APK/OBB/IPA. diff --git a/tests/update.TestCase b/tests/update.TestCase index bebab3f0..37e4be9b 100755 --- a/tests/update.TestCase +++ b/tests/update.TestCase @@ -165,6 +165,7 @@ class UpdateTest(unittest.TestCase): apps['eu.siacs.conversations']['Builds'] = [build_conversations] fdroidserver.update.insert_localized_app_metadata(apps) + fdroidserver.update.ingest_screenshots_from_repo_dir(apps) appdir = os.path.join('repo', 'info.guardianproject.urzip', 'en-US') self.assertTrue( @@ -278,6 +279,7 @@ class UpdateTest(unittest.TestCase): knownapks = fdroidserver.common.KnownApks() apks, cachechanged = fdroidserver.update.process_apks({}, 'repo', knownapks, False) fdroidserver.update.insert_localized_app_metadata(apps) + fdroidserver.update.ingest_screenshots_from_repo_dir(apps) fdroidserver.update.apply_info_from_latest_apk(apps, apks) app = apps['info.guardianproject.urzip'] self.assertIsNone(app.Name)