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 को "+" । यह मोपिडी वर्तमान में जो करता है उससे अलग है - पथ विभाजक केवल + साथ प्रतिस्थापित किए जाते हैं यदि हम पथ के लेबल के रूप में वास्तविक फ़ाइल पथ का पुन: उपयोग कर रहे हैं। मैं एक कारण के बारे में सोचने में सक्षम नहीं था कि पथ लेबल में स्लैश क्यों नहीं हो सकते थे, इसलिए मुझे लगता है कि बिना शर्त प्रतिस्थापन करने के लिए यह समझ में आता है।

संदर्भ: https://github.com/mopidy/mopidy/blob/develop/mopidy/file/library.py#L133

क्या यह मुद्दा अभी भी खुला है? मैं इस पर काम कर सकता हूं अगर कोई मुझे दिशा प्रदान करे। मैं एक नया योगदानकर्ता हूं।

हां। मुझे लगता है कि मूल पोस्ट में दिशा है लेकिन कृपया पूछें कि क्या आपके पास कोई विशिष्ट प्रश्न है।

@kingosticks कॉन्फ़िगरेशन पार्सिंग सिस्टम को और अधिक शक्तिशाली बनाने के मेरे थोड़े बड़े प्रस्ताव के बारे में आप क्या सोचते हैं?

क्षमा करें, हाँ, चूंकि फ़ाइल 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

विशेष रूप से:

• String और List प्रकारों को अपडेट करें, जैसा कि #1966 . में प्रलेखित है
• ऊपर बताए अनुसार Pair प्रकार लागू करें

आपको इस नई कार्यक्षमता के लिए यहां परीक्षण भी जोड़ने चाहिए:

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

एक बार यह हो जाने के बाद, आप फ़ाइल एक्सटेंशन को अपडेट कर सकते हैं। __init__.py में कॉन्फ़िगरेशन परिभाषा को ऊपर दिए गए मेरे उदाहरण की तर्ज पर अद्यतन किया जाना चाहिए। फिर आपको library.py में मीडिया निर्देशिका पार्सिंग कोड को अत्यधिक सरल बनाने में सक्षम होना चाहिए।

कृपया हमें बताएं कि क्या आपको अधिक सहायता या सलाह की आवश्यकता है, या यदि मैंने जो कुछ भी लिखा है उसका कोई मतलब नहीं है।

@ djmattyg007 स्पष्टीकरण के लिए बहुत बहुत धन्यवाद मैं एक मसौदा पीआर तैयार करूंगा और आपको वहां टैग करूंगा।

@ abid1998 आप इन अपडेट के साथ कैसे जा रहे हैं? क्या आपको किसी सहायता की आवश्यकता है?