Reply To: desktop-menu update

[ileMember]

Hello Dave
one in 15625 and 1 in 15625.
when compliant form
or consider the configuration malconformed, not compliant
forty to the sixth power
the odds of this being true
one in 40 to power 6 and 1 in 15625. (?)
line 526 if Configuration.separators and opts.separators == [”, ”, ‘[‘, ‘]’, ‘(‘, ‘)’]:exact

one in four the odds of this be true (?)
line 526 if Configuration.separators and opts.separators:exist

These conditions in .new imply a wish to prioritize command line options.

    def __init__(self, user_info):
    
        if "*" not in Configuration.text_length: self.text_length = Configuration.text_length
        if "*" not in opts.text_length: self.text_length = opts.text_length
        
        if Configuration.category_filter and opts.category_filter == "ngc":
            self.category_filter = Configuration.category_filter
        else:
            self.category_filter = opts.category_filter
            
        if Configuration.text_order and opts.text_order == "n":
            self.text_order = list(Configuration.text_order)
        else:
            self.text_order = list(opts.text_order)
            
        if Configuration.separators and opts.separators == ['', '', '[', ']', '(', ')']:
            self.separators = list(Configuration.separators)
        else:
            self.separators = opts.separators

changed to this

        if Configuration.text_length >= "1": self.text_length = Configuration.text_length
        if opts.text_length >= "1": self.text_length = opts.text_length
        
        if opts.category_filter != "ngc":
            self.category_filter = opts.category_filter
        else:
            self.category_filter = Configuration.category_filter
            
        if opts.text_order != "n":
            self.text_order = list(opts.text_order)
        else:
            self.text_order = list(Configuration.text_order)
            
        if opts.separators != ['', '', '[', ']', '(', ')']:
            self.separators = opts.separators
        else:
            self.separators = list(Configuration.separators)

The first two are MOVED.
They can over ride the icewm over ride of text length,
first two lines moved to the area of lines 443 and 502 now

                                separators = self.build_text("pop", "", separators)

                    if Configuration.text_length >= "1":
                        self.text_length = Configuration.text_length
                    if opts.text_length >= "1":
                        self.text_length = opts.text_length
                        text = text[:int(self.text_length)]
                    if wm not in ("icewm") and hasattr(self, 'text_length'):
                        text = text[:int(self.text_length)]
                    if wm == "fluxbox":
                        text=text.replace(')',r'\)')
                if wm in ('icewm') and opts.icons_only:
                    text = " "