Mopidy: 파일 백엔드의 media_dirs 설정에 있는 추가 공백은 무시되지 않습니다.

사용자가 지정한 경로 이름에 대해 어떠한 가정도 해서는 안 된다고 생각합니다. 그들이 합법적으로 공백 문자로 끝나는 경로를 가지고 있다면, 그들은 그것을 현명하게 사용할 수 없을 것입니다.

하지만 오류 메시지에서 경로를 더 잘 나타내는 것은 확실히 좋은 생각입니다.

나는 그것을 시도하지 않았지만 우리의 Path 구성 유형은 strip() 값이 될 것이므로 이것이 동일해야 한다고 생각했습니다. 즉, 현재 Mopidy 구성 파일의 공백으로 끝나는 경로를 설정할 수 없습니다. 나는 당신의 요점에 동의하지 않지만 구성 파일의 줄 끝에 우발적으로 공백이 있는 것이 공백으로 끝나는 경로보다 훨씬 더 일반적이라고 상상할 수 있습니다. 따라서 이것은 실제로 의도적일 수 있으며 확실하지 않을 수 있습니다.

그것은 의미가 있습니다.

#1966에 대한 생각의 일환으로 구성 처리 개선 아이디어가 File 확장에서도 활용될 수 있다는 것을 깨달았습니다. 해당 문제에 대해 제공한 예제에서 확장:

class Pair(ConfigValue):
    def __init__(self, optional=False, optional_pair=False, separator="|", subtypes=None):
        self._required = not optional
        self._optional_pair = optional_pair
        self._separator = separator
        if subtype:
            self._subtypes = subtype
        else:
            self._subtypes = (String(), String())

    def deserialize(self, value):
        raw_value = decode(value).strip()
        validators.validate_required(raw_value, self._required)

        if self._separator in raw_value:
            value = value.split(self._separator, 1)
        elif self._optional_pair:
            value = (raw_value, raw_value)
        else:
            raise ValueError("must have separator")

        return (self._subtypes[0].deserialize(value[0]), self._subtypes[1].deserialize(value[1]))

    def serialize(self, value, display=False):
        return "{0}{1}{2}".format(
            self._subtypes[0].serialize(value, display),
            self._separator,
            self._subtypes[1].serialize(value, display),
        )

그런 다음 File 확장에 대해 다음과 같이 작성할 수 있습니다.

from mopidy import config
import os

schema["media_dirs"] = config.List(
    optional=True,
    subtype=config.Pair(
        optional=False,
        optional_pair=True,
        subtypes=(
            config.Path(),
            config.String(transformer=lambda x: x.replace(os.sep, "+")),
        ),
    ),
)

이렇게 하면 File 확장의 액터 코드 내에서 이러한 작업을 처리할 필요가 없으며 Mopidy가 시작되기 전에 구성이 완전히 검증됩니다.

제쳐두고, 위의 코드 샘플은 무조건 os.sep 를 "+" 바꿉니다. 이것은 현재 Mopidy가 하는 것과 다릅니다. 경로 구분 기호는 실제 파일 경로를 경로 레이블로 재사용하는 경우에만 + 대체됩니다. 경로 레이블에 슬래시를 사용할 수 없는 이유를 생각할 수 없었으므로 무조건 대체를 수행하는 것이 합리적이라고 생각합니다.

참조: https://github.com/mopidy/mopidy/blob/develop/mopidy/file/library.py#L133

이 문제가 아직 열려 있습니까? 누군가 나에게 방향을 제시하면 나는 그것을 할 수 있다. 저는 새로운 기여자입니다.

예. 원래 글에 방향이 있는 것 같은데 구체적인 질문이 있으면 질문해주세요.

@kingosticks 구성 구문 분석 시스템을 더 강력하게 만드는 약간 더 큰 제안에 대해 어떻게 생각하십니까?

죄송합니다. 예, File은 Mopidy와 함께 번들로 제공되므로 방향도 괜찮습니다. 수정과 테스트 모두에서 더 복잡합니다. @abid1998 이 된다면 좋은 것 같습니다.

@kingosticks @ djmattyg007
내가 이해한 바에 따르면 이 파일을 변경해야 합니까?
https://github.com/mopidy/mopidy/blob/HEAD/mopidy/file/__init__.py
https://github.com/mopidy/mopidy/blob/HEAD/mopidy/file/library.py
그래서 오른쪽에서 공백을 제거해야 | 캐릭터 ? 아니면 공백이 있으면 유용한 오류 메시지를 표시해야 합니까?

@abid1998 이 파일을 변경하여 시작하려고 합니다.

https://github.com/mopidy/mopidy/blob/develop/mopidy/config/types.py

구체적으로 특별히:

• #1966에 설명된 대로 String 및 List 유형을 업데이트합니다.
• 위에서 언급한 대로 Pair 유형을 구현합니다.

또한 여기에 이 ​​새로운 기능에 대한 테스트를 추가해야 합니다.

https://github.com/mopidy/mopidy/blob/develop/tests/config/test_types.py

완료되면 파일 확장자를 업데이트할 수 있습니다. __init__.py 의 구성 정의는 위의 예와 같이 업데이트되어야 합니다. 그러면 library.py 의 미디어 디렉토리 구문 분석 코드를 대폭 간소화할 수 있습니다.

더 많은 도움이나 조언이 필요하거나 제가 작성한 내용이 이해가 되지 않는 경우 알려주십시오.

@djmattyg007 설명

@abid1998 이번 업데이트는 어떻게 하고 계신가요? 도움이 필요하십니까?