×
Please submit new Bug Reports on GitHub: github.com/Jensen-Technologies/component-creator-issues/issues
1 month plan
9 år 7 månader sedan #725
av Cedric
Svar från Cedric i ämnet 1 month plan
? 
$app = JFactory::getApplication();
$editId = JFactory::getApplication()->input->getInt('id', null, 'array');
should be:
$app = JFactory::getApplication();
$editId = $app->input->getInt('id', null, 'array');
$app = JFactory::getApplication();
$editId = JFactory::getApplication()->input->getInt('id', null, 'array');
should be:
$app = JFactory::getApplication();
$editId = $app->input->getInt('id', null, 'array');
Be Logga in eller Skapa ett konto ansluta till konversationen.
9 år 7 månader sedan - 9 år 7 månader sedan #726
av David Villena
Svar från David Villena i ämnet 1 month plan
Hi Rick.
First of all, thank you very much for your feedback. Component Creator wouldn't be so awesome without the huge team of testers that you guys are
You're right about the code formatting; we're so focused on features, functionality and bug fixes that sometime we forget that code must look pretty and well-aligned for readability. We have a job on the queue to revise all the code generated and fix all format issues.
However, we can't agree about indentation: Joomla! coding standards , are based (with modifications) on PSR-2 Coding Style Guide , and there it is written:
2.4. Indenting
Code MUST use an indent of 4 spaces, and MUST NOT use tabs for indenting.
And we've solved the $app unused thing.
Best regards,
David.
First of all, thank you very much for your feedback. Component Creator wouldn't be so awesome without the huge team of testers that you guys are
You're right about the code formatting; we're so focused on features, functionality and bug fixes that sometime we forget that code must look pretty and well-aligned for readability. We have a job on the queue to revise all the code generated and fix all format issues.
However, we can't agree about indentation: Joomla! coding standards , are based (with modifications) on PSR-2 Coding Style Guide , and there it is written:
2.4. Indenting
Code MUST use an indent of 4 spaces, and MUST NOT use tabs for indenting.
About the Module Builder, on the Configuration view of your component you can find a checkbox with the label "Include Module" that, if checked, makes Component Creator to generate the structure of an empty module, that you can fill with the functionality that you need for it. Screenshot is attached.N.b.: Using only spaces, and not mixing spaces with tabs, helps to avoid problems with diffs, patches, history, and annotations. The use of spaces also makes it easy to insert fine-grained sub-indentation for inter-line alignment.
And we've solved the $app unused thing.
Best regards,
David.
Be Logga in eller Skapa ett konto ansluta till konversationen.
Sidan laddades på: 0.056 sekunder