Module name reference

Comments

• admin

  September 2011 edited 10:24PM

  Yep... thanks for the report. I'll fix that.
• admin

  September 2011 edited 10:24PM

  That has been fixed here:
  https://github.com/daylightstudio/FUEL-CMS/commit/39516d209661225aac3a5ceec4a0dd1f96c6ec86
• ancarius

  September 2011 edited 10:24PM

  I see the module name is now humanized. But in my case the only way I managed to specify the keys is with a prefix (ie. $config['modules']['bio_categories']) which get still returns the prefix in the instruction.
  
  For example in:
  
  $config['modules']['bio_categories'] = array(
  'module_name' => 'Categories',
  'module_uri' => 'bio/categories',
  'model_name' => 'bio_categories_model',
  'model_location' => 'bio',
  'table_headers' => array(
  'id',
  'name',
  'precedence',
  'published'
  ),
  'nav_selected' => 'bio/categories',
  'instructions' => 'Here you can manage categories.'
  );
  
  ...shouldn't the Instruction read the lowercase value of 'module_name', as I suspect that most people would use a sensible name that would easily be understandable in the instructions.
• ancarius

  September 2011 edited September 2011

  If I remove the prefix I end up with an error message:
  
  Severity: Notice --> Undefined property: Module::$model_name \wamp\www\fpse\fuel\modules\fuel\controllers\module.php 79
  
  Am I possibly doing something wrong? Is there someplace I should specify my module prefix?
• admin

  September 2011 edited 10:24PM

  I think I see the issue and believe I have a fix locally which I can push soon. The fix was to add the following to the fuel/modules/fuel/libraries/Fuel_modules.php file around line 127 and then change the default value for instructions to NULL on line 85:
  // set instructions if (empty($return['instructions'])) { $return['instructions'] = lang('module_instructions_default', strtolower($return['module_name'])); }
  
  Try this out and see if that fixes your issue.