5th Meeting Joomla GSoC 2016
Date: 28st Jun 2016 on Glip Chat
Time: 14:30 (UTC / GMT)
Meeting was attended by: Shubham Rajput, Gunjan Patel, Peter Martin
Congratulate Shubham on passing his midterm evaluation and his thoughts about it
Peter and Gunjan congratulate Shubham for passing his midterm evaluation. Here is the words from Shubham what he has experienced and how he is feeling after passing midterm evaluation.
“It’s been an awesome journey till now with joomla and you guys
Since the coding period started It’s been a different experience each week. Starting week I was nervous when i first dived in the component architecture. But you guys were there whenever i faced difficulty or when i went in wrong direction. Weekly meeting helped a lot.
Getting a new target and completing it gave more confidence to me with each passing week. In this coding period I also learnt many new things,went through some sleepless nights working on code and understanding the architecture of joomla more and more. Also went through mixed phases(pressure,relief,yay,nay). These are normal whenever we do coding
But in the end after passing the mid term evaluation I think that all of the above was worth it.
GSoC and involving with Joomla project is a experience which not everyone gets easily and it feels awesome to be a part of it.
Today i received mail from Google regarding your feedback on my work. After reading it I was really happy that i had not let you down. Kudos to all of us.”
Presentation & Review on the previous week work
Previous week the first target was to make the menu link tab work. The problem occurring before was that after filling all the necessary information in the article edit view and menu link tab and clicking ‘save’ button results only in the saving of the article but no such save occurs in __menus table for menu item. The menu link tab was not working as it was intended.
Gunjan suggested to go through JModelAdmin functions and find the solution. After going through the JModelAdmin addtablepath() function solved the issue. The menu link tab started to work and menu item was being created. After that I was using database queries in order to get the component_id and article_id for menu data variable. Gunjan pointed out my mistake that controller is not the place to use queries and suggested alternate method. After that I used functions to retrieve both component_id and article_id which can be seen in this commit: https://github.com/joomla-projects/gsoc16_menu-for-article/pull/6/commits/be4dfb055e5d3186585cc228611d1b0e88e97049
Then after I improved the coding standard using camelCase notation which I missed, removing redundancy in the code and making changes that made code easily readable.
This marked the midterm evaluation target as success. Cheers to both my mentors and joomla community .
After following the advice of Peter and Gunjan I installed XDebug to debug the code more easily. I also went through the article that were suggested by Gunjan and Peter to understand hierarchical concept in mysql and understand what ‘lft’ and ‘rgt’ column meant in __content table.
Another resource was suggested by Gunjan to go through and understand the architecture of com_content coding. I went through https://issues.joomla.org/tracker/joomla-cms/10851
And understood what steps were used to improve the code style and reducing the code length using much more simpler method. I asked some doubts regarding the commit from @mbabker whose PR I was going through.
Timeline & Task for next week
2. To look through both methods, there advantages/disadvantages and discuss it with mentors, also look forward for new idea out of the box.
3. The current implementation sets 'language' => '*' in the newly created menu item. This might be a problem with multilingual articles on multilingual websites. New task is to use article’s language code to set the language code for the menu item.
Shubham asked about PHPCS is working fine but when he tried to run it on administrator\components\com_content\views\article\tmpl\edit.php file it is showing 44 errors only for this files and Peter and Gunjan suggested to ignore them for now as they are due to the Joomla code style seems not so happy with inline control statements like:
Resources to read & example suggested
Next meeting date & timing
Date: 5th July 2016 on Glip Chat
Time: 14:30 (UTC / GMT)