Howdy everyone! Here’s the third edition of the PrestaShop Core Weekly.

As a reminder, the Core Weekly posts highlight some of the more important happenings in and around the Core of PrestaShop.

This post is focused on the activity from March 9th to March 15th of 2015.

Do you have ideas on how to improve the Core Weekly posts? Let me know in the comments!

Milestones

Some cool things are happening for our community and the overall improvement of the codebase. It’s not quite ready yet, but soon you’ll see it all, and you’ll be able to participate!

One thing to notice: PrestaShop developers now do all their work using Pull Requests (PR): no more direct commits to the codebase, now you can see where new code comes from, and comment on the PR! Consequently, the “Notable commits” section of this Core Weekly report is not needed anymore :)

Code changes

Now, onto last week’s code changes…

Notable accepted pull requests

  • 2653: PHPDoc types for class members that are classes. Thanks gskema!
  • 2652: FO: Fixing missing data-title attribute and translations. Thanks dkarvounaris!
  • 2637: BO : fix count optimizations, by jocel1.
  • 2598: BO : Optimize cart rules loading & add a search filter field, by jocel1.
  • 2589: FO VAT field is shown when change country. Thanks prestarocket!
  • 2582: Add hook actionPDFInvoiceRender. Thanks cedricfontaine!
  • 2581: Hook actionPDFInvoiceRender consistency. Thanks cedricfontaine!
  • 2568: CORE: Debug tools for printing into PHP error logs. Thanks gskema!

Find all PRs closed last week here.

Developers, do you have code suggestions, improvement ideas, typos you want to fix? Make your own Pull Request and become part of the PrestaShop contributors! See the contribution guide here.

Do you want a first project to contribute with? Give us feedback on this PR. Thank you!

Notable closed Forge tickets

  • 2614: BO Fix #PSCSX-274 : Cumulative specific rules fixed, by Shudrum. A ticket that dates back from November 2013, woohoo!
  • 2648: Do not load the full category tree if it’s not needed (PSCSX-4591), by jocel1.
  • 2608: BO : Fix bug #PSCSX-4733, threads statuses not translated, by gRoussac.
  • 2601: FO : Fix css bug #PSCSX-4757, display nav on small screens, by gRoussac.

Find all Forge tickets closed last week here: click here!

Do you have an issue or idea but you are not a developer! Create a ticket on the Forge, in the PSCSX project!

That’s it for last week! This was the third edition of Core Weekly, so again, if you have suggestions or see anything that I missed, please comment!