Download as
Browse Code »

Commit 8f8240bd6329e1e12cdda30fca2dfab451530517

Authored by Sergio Oliveira
1 parent 854a1dad
Exists in master and in 25 other branches 1.12.x, add_requests_as_dep, celery_service, change-passwd-signal, colab-vcard, colab_search, colab_tag, detach_super_archives, fix-dashboard, fix-message-preview, fix_header, go_to_profile_panel, mobile_user_scalable, paginate-threads, profile_integration, remove-gitlab-plugin, removing_namespace, search_block, search_filters, split_screen, startplugin, timestamp_plugins, translation_review, user_regex, validate-passwd

Better messaging when failing 

Signed-off-by: Sergio Oliveira <sergio@tracy.com.br>
Showing 1 changed file with 7 additions and 2 deletions   Show diff stats
colab/utils/conf.py
  Diff comments   View file @8f8240b
... ... @@ -98,15 +98,20 @@ def load_colab_apps():
98 98 app_name = ""
99 99 file_module = file_name.split('.')[0]
100 100  
101   - logger.info('Loaded plugin settings: %s%s', plugins_dir, file_name)
102   - py_settings_d = _load_py_file(file_module, plugins_dir)
  101 + logger.info('Loading plugin settings: %s%s', plugins_dir, file_name)
103 102  
104 103 if os.path.isdir(os.path.join(plugins_dir, file_name)):
  104 + py_settings_d = _load_py_file(file_module, plugins_dir)
105 105 app_name = file_name
106 106  
107 107 elif file_name.endswith('.py'):
  108 + py_settings_d = _load_py_file(file_module, plugins_dir)
108 109 app_name = py_settings_d.get('name', '').split('.')[-1]
109 110  
  111 + else:
  112 + logger.info("Not a plugin config: %s", file_name)
  113 + continue
  114 +
110 115 if not app_name:
111 116 logger.warning("Plugin missing name variable (%s)", file_name)
112 117 continue
... ...