missing </fieldset> in admin\views\helloworld\tmpl\edit.php

Looking at line 27 in the file admin\views\helloworld\tmpl\edit.php should look like this, as shown in the tutorial.

</fieldset>

The file in the zipfile is missing this line </fieldset> (line 27)

Another minor detail is that the page on the website shows blocks of <?php ... ?> whereas the file as the php open and close per line.

Does not influence the working. But is good to know when comparing your code. I use winmerge to compare my files with those in the zip file. Hoping to find that typo that prevents my version from running as it should ;-(

Modifications that would make the tutorial clearer

I think the following 2 modifications would make this section of the tutorial clearer:

Earlier in the tutorial, in the file site/models/helloworld.php $msg was used for the message, and here is is $item. I think it would be helpful to stay with the same variable name throughout the tutorial.

In the file admin/tables/helloworld.php, the comment about deprecated code is confusing. It might be better if it read

// loadJSON is @deprecated 12.1 (Joomla! 3) Use loadString...

so it is more obvious that the current code is correct for this version (2.5) but not the future version (3). Not everyone knows what the 12.1 is referring to and it looks like it is saying the code the tutorial is telling you to use is deprecated.

Changes to code created in previous articles

Reading through this entire series I have had a hard time determining what is added when a section of code is shown that says "add these lines to file_name".

It would help to indicate the changes in some way (possibly highlighting).

I don't know if the writers of the series even realised this, plus some of the wording might indicate creating the entire code snippet in a blank file. It would be a great help if you wanted to update them, but I would suggest you start with 3.x version. 2.5 will reach EOL Dec of 2014. Thanks! Tom Hutchison (talk) 20:28, 10 July 2014 (CDT)