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

Code before:
        if sts is not None:
            context['signatureTimestamp'] = sts
        return {
            'playbackContext': {
                'contentPlaybackContext': context,
            },
            **cls._get_checkok_params(),
        }

    def _get_config_po_token(self, client):
        po_token_strs = self._configuration_arg('po_token', [], ie_key=YoutubeIE, casesense=True)
        for token_str in po_token_strs:
            if '+' not in token_str:
                self.report_warning('Invalid po_token configuration format. Expected "client+po_token"')
                continue
            po_token_client, po_token = token_str.split('+')
            if not client or po_token_client == client:
                return po_token

    def fetch_po_token(self, client='web', visitor_data=None, data_sync_id=None, player_url=None, **kwargs):
        # PO Token is bound to visitor_data / Visitor ID when logged out. Must have visitor_data for it to function.
        if not visitor_data and not self.is_authenticated and player_url:
            self.report_warning(
                f'Unable to fetch PO Token for {client} client: Missing required Visitor Data. '
                f'You may need to pass Visitor Data with --extractor-args youtube:visitor_data=XXX',
            )
            return

        config_po_token = self._get_config_po_token(client)
        if config_po_token:
            # PO token is bound to data_sync_id / account Session ID when logged in. However, for the config po_token,
            # if using first channel in an account then we don't need the data_sync_id anymore...
            if not data_sync_id and self.is_authenticated and player_url:
                self.report_warning(
                    f'Got a PO Token for {client} client, but missing Data Sync ID for account. Formats may not work.'
                    f'You may need to pass a Data Sync ID with --extractor-args youtube:data_sync_id=XXX',
                )

            return config_po_token

        # Require PO Token if logged in for external fetching
        if not data_sync_id and self.is_authenticated and player_url:
            self.report_warning(
                f'Unable to fetch PO Token for {client} client: Missing required Data Sync ID for account. '
                f'You may need to pass a Data Sync ID with --extractor-args youtube:data_sync_id=XXX',
            )
            return

        return self._fetch_po_token(
            client=client,
            visitor_data=visitor_data,

Review comment (refactor):
```suggestion
                'You may need to pass Visitor Data with --extractor-args youtube:visitor_data=XXX')
```
and same below?

Code after:
            **cls._get_checkok_params(),
        }

    def _get_config_po_token(self, client):
        po_token_strs = self._configuration_arg('po_token', [], ie_key=YoutubeIE, casesense=True)
        for token_str in po_token_strs:
            po_token_client, sep, po_token = token_str.partition('+')
            if not sep:
                self.report_warning(
                    f'Invalid po_token configuration format. Expected "client+po_token", got "{token_str}"', only_once=True)
                continue
            if po_token_client == client:
                return po_token

    def fetch_po_token(self, client='web', visitor_data=None, data_sync_id=None, player_url=None, **kwargs):
        # PO Token is bound to visitor_data / Visitor ID when logged out. Must have visitor_data for it to function.
        if not visitor_data and not self.is_authenticated and player_url:
            self.report_warning(
                f'Unable to fetch PO Token for {client} client: Missing required Visitor Data. '
                f'You may need to pass Visitor Data with --extractor-args "youtube:visitor_data=XXX"')
            return

        config_po_token = self._get_config_po_token(client)
        if config_po_token:
            # PO token is bound to data_sync_id / account Session ID when logged in. However, for the config po_token,
            # if using first channel in an account then we don't need the data_sync_id anymore...
            if not data_sync_id and self.is_authenticated and player_url:
                self.report_warning(
                    f'Got a PO Token for {client} client, but missing Data Sync ID for account. Formats may not work.'
                    f'You may need to pass a Data Sync ID with --extractor-args "youtube:data_sync_id=XXX"')

            return config_po_token

        # Require PO Token if logged in for external fetching
        if not data_sync_id and self.is_authenticated and player_url:
            self.report_warning(
                f'Unable to fetch PO Token for {client} client: Missing required Data Sync ID for account. '
                f'You may need to pass a Data Sync ID with --extractor-args "youtube:data_sync_id=XXX"')
            return

        return self._fetch_po_token(
            client=client,
            visitor_data=visitor_data,
            data_sync_id=data_sync_id,
            player_url=player_url,
            **kwargs,
        )

    def _fetch_po_token(self, client, visitor_data=None, data_sync_id=None, player_url=None, **kwargs):
        """External PO Token fetch stub"""