components section is not completed in "config set" of trac-admin

[Jun Omae]

In config subcommands, existent sections and options are only completed. Fresh trac.ini doesn't have [components] section. Then, the components section is not completed.

I think defined sections and options should be completed in config set command even if not existent.

[Ryan J Ollos]

How about this fix?:

trac/config.py

@@ -953 +953 @@
                self._complete_config, self._do_set)
 
     def _complete_config(self, args):
+        sections = set(ConfigSection.get_registry(self.env.compmgr))
+        options = set(Option.get_registry(self.env.compmgr))
         if len(args) == 1:
-            return self.config.sections()
+            section_names = sections | set(i[0] for i in options)
+            return section_names
         elif len(args) == 2:
-            return [name for (name, value) in self.config[args[0]].options()]
+            option_names = set(i[1] for i in options if i[0] == args[0])
+            return option_names
 
     def _do_get(self, section, option):
         if not self.config.has_option(section, option):

$ trac-admin t
Welcome to trac-admin 1.3.3.dev0
Interactive Trac administration console.
Copyright (C) 2003-2017 Edgewall Software

Type:  '?' or 'help' for help on commands.

Trac [/Users/rjollos/Documents/Workspace/trac-dev/teo-rjollos.git/t]> config set
attachment               milestone                roadmap
browser                  milestone-groups         search
changeset                mimeviewer               sqlite
components               notification             svn
header_logo              notification-subscriber  ticket
inherit                  project                  ticket-custom
intertrac                pygments-lexer           ticket-workflow
interwiki                query                    timeline
logging                  report                   trac
mainnav                  repositories             versioncontrol
metanav                  revisionlog              wiki
Trac [/Users/rjollos/Documents/Workspace/trac-dev/teo-rjollos.git/t]> config set project
admin           descr           icon            url
admin_trac_url  footer          name
Trac [/Users/rjollos/Documents/Workspace/trac-dev/teo-rjollos.git/t]> config set inter
intertrac  interwiki

[Ryan J Ollos]

Proposed changes in [77cb74b9d/rjollos.git].

[Jun Omae]

I tried. Sections and options which are not declared with ConfigSection and Option in trac.ini are not listed.

Trac [/var/trac/1.2-sqlite]> config set foo-section bar-option 42
Trac [/var/trac/1.2-sqlite]> config remove foo
                                              ^ Enter TAB key but none is completed

Also, svn:externals section is declared in svn_prop.py but not listed.

[Ryan J Ollos]

Replying to Jun Omae:

Also, svn:externals section is declared in svn_prop.py but not listed.

I find that it is listed when the component is enabled. Is the component enabled in your environment?

Trac [/Users/rjollos/Documents/Workspace/trac-dev/teo-rjollos.git/env]> config get components tracopt.versioncontrol.*
enabled
Trac [/Users/rjollos/Documents/Workspace/trac-dev/teo-rjollos.git/env]> config get
attachment               metanav                  roadmap
browser                  milestone                search
changeset                milestone-groups         sqlite
components               mimeviewer               svn
git                      notification             svn:externals
gitweb-repositories      notification-subscriber  ticket
header_logo              project                  ticket-custom
inherit                  pygments-lexer           ticket-workflow
intertrac                query                    timeline
interwiki                report                   trac
logging                  repositories             versioncontrol
mainnav                  revisionlog              wiki

[Jun Omae]

Replying to Ryan J Ollos:

Replying to Jun Omae:

Also, svn:externals section is declared in svn_prop.py but not listed.

I find that it is listed when the component is enabled. Is the component enabled in your environment?

Ah. Missing tracopt.versioncontrol.* = enabled in my environment. The svn:externals is listed after enabling the components. Thanks.

[Ryan J Ollos]

Replying to Jun Omae:

I tried. Sections and options which are not declared with ConfigSection and Option in trac.ini are not listed.

Trac [/var/trac/1.2-sqlite]> config set foo-section bar-option 42
Trac [/var/trac/1.2-sqlite]> config remove foo
                                              ^ Enter TAB key but none is completed

I reconsidered how to solve the issue: [be1bf0044/rjollos.git].

In most cases we don't want the empty sections to be returned from Configuration.sections(). I added an optional parameter empty, which defaults to False and determines whether entries from the ConfigSection registry are included.

[Ryan J Ollos]

Committed to 1.2-stable in r16231, merged to trunk in [16232,16234].

I noticed the output was sorted without an explicit sort action in console.py​. I tried debugging in the code and could not see where the output was being formatted. My best guess is that it's happening in readline.

Trac [/Users/rjollos/Documents/Workspace/trac-dev/tracenvs/proj-1.2]> config get
account-manager            mainnav                    spam-filter
attachment                 metanav                    sqlite
authz_policy               milestone                  svn
autowikify                 milestone-groups           svn:externals
browser                    mimeviewer                 ticket
changeset                  multiselectfield           ticket-custom
childtickets               notification               ticket-workflow
components                 notification-subscriber    ticketstats
discussion                 plantuml                   timeline
extra-permissions          project                    trac
git                        pygments-lexer             versioncontrol
gitweb-repositories        query                      wiki
header_logo                report                     wiki-notification
hg                         repositories               wikiextras
inherit                    revisionlog                wikiextras-custom-phrases
intertrac                  roadmap                    wikiextras-smileys
interwiki                  search                     wikiextras-symbols
logging                    simple-multi-project

I noticed that the output is not sorted when using ​pypi:pyreadline:

Trac [C:\Users\Ryan Ollos\My Documents\Workspace\proj-1.2]> config set
wiki                    notification            search
changeset               gitweb-repositories     interwiki
sqlite                  attachment              logging
authz_policy            mimeviewer              ticket-custom
roadmap                 milestone-groups        trac
header_logo             mainnav                 repositories
intertrac               timeline                inherit
versioncontrol          extra-permissions       project
svn:externals           metanav                 pygments-lexer
query                   milestone               components
notification-subscriber report                  ticket-workflow
revisionlog             ticket                  browser
git                     svn

I tried sorting the values returned by ConfigurationAdmin._complete_config, but it didn't seem to make a difference.

Anyway, I don't think it's too important, but if anyone determines the cause we can handle it in a new ticket.