Repository: yt-dlp/yt-dlp
File: yt_dlp/extractor/allstar.py (Python)

Code before:
        }
    }''' % (_FIELDS, _EXTRA_FIELDS),
    'montages': '''query ($page: Int!, $user: String!) {
        videos: montages(search: createdDate, page: $page, user: $user) {
            data { %s }
        }
    }''' % _FIELDS,
    'mobile clips': '''query ($page: Int!, $user: String!) {
        videos: clips(search: createdDate, page: $page, user: $user, mobile: true) {
            data { %s %s }
        }
    }''' % (_FIELDS, _EXTRA_FIELDS),
}

_PAGESIZE = 10


def _media_url_or_none(path):
    return urljoin('https://media.allstar.gg/', str_or_none(path))


def _profile_url_or_none(path):
    return urljoin('https:/allstar.gg/u/', str_or_none(path))


def _parse_video_data(video_data):
    return traverse_obj(video_data, {
        'id': ('_id', {str_or_none}),
        'display_id': ('shareId', {str_or_none}),
        'title': ('clipTitle', {str_or_none}),
        'url': ('clipLink', {_media_url_or_none}),
        'thumbnail': ('clipImageThumb', {_media_url_or_none}),
        'duration': ('clipLength', {int_or_none}),
        'filesize': ('clipSizeBytes', {int_or_none}),
        'timestamp': ('createdDate', {int_or_none}),
        'uploader': ('username', {str_or_none}),
        'uploader_id': ('user', '_id', {str_or_none}),
        'uploader_url': ('user', '_id', {_profile_url_or_none}),
        'view_count': ('views', {int_or_none}),
        'categories': ('game', {str_or_none}),
    })


def _set_webpage_url(info_dict):
    video_id = info_dict.get('id')
    video_url = info_dict.get('url')

    if video_url is None or video_id is None:
        return info_dict

    base_name = 'clip' if '/clips/' in video_url else 'montage'

Review comment (suggestion):
I would move `_media_url_or_none` and `_profile_url_or_none` inside of `_parse_video_data` and then `_parse_video_data` inside of `AllstarBaseIE` as a staticmethod.

Code after:
    'Montages': '''query ($page: Int!, $user: String!) {
        videos: montages(search: createdDate, page: $page, user: $user) {
            data { %s }
        }
    }''' % _FIELDS,
    'Mobile Clips': '''query ($page: Int!, $user: String!) {
        videos: clips(search: createdDate, page: $page, user: $user, mobile: true) {
            data { %s %s }
        }
    }''' % (_FIELDS, _EXTRA_FIELDS),
}


class AllstarBaseIE(InfoExtractor):
    @staticmethod
    def _parse_video_data(video_data):
        def media_url_or_none(path):
            return urljoin('https://media.allstar.gg/', path)

        info = traverse_obj(video_data, {
            'id': ('_id', {str}),
            'display_id': ('shareId', {str}),
            'title': ('clipTitle', {str}),
            'url': ('clipLink', {media_url_or_none}),
            'thumbnails': (('clipImageThumb', 'clipImageSource'), {'url': {media_url_or_none}}),
            'duration': ('clipLength', {int_or_none}),
            'filesize': ('clipSizeBytes', {int_or_none}),
            'timestamp': ('createdDate', {functools.partial(int_or_none, scale=1000)}),
            'uploader': ('username', {str}),
            'uploader_id': ('user', '_id', {str}),
            'view_count': ('views', {int_or_none}),
        })

        if info.get('id') and info.get('url'):
            basename = 'clip' if '/clips/' in info['url'] else 'montage'
            info['webpage_url'] = f'https://allstar.gg/{basename}?{basename}={info["id"]}'

        info.update({
            'extractor_key': AllstarIE.ie_key(),
            'extractor': AllstarIE.IE_NAME,
            'uploader_url': urljoin('https://allstar.gg/u/', info.get('uploader_id')),
        })

        return info

    def _call_api(self, query, variables, path, video_id=None, note=None):
        response = self._download_json(
            'https://a1.allstar.gg/graphql', video_id, note=note,
            headers={'content-type': 'application/json'},
            data=json.dumps({'variables': variables, 'query': query}).encode())