PrestaShop Core Weekly - Week 46 of 2019
An inside look at the PrestaShop codebase
This edition of the Core Weekly report highlights changes in PrestaShop’s core codebase from Monday 11th to Sunday 17th of November 2019.
A quick update about PrestaShop’s GitHub issues and pull requests:
- 62 new issues have been created in the project repositories;
- 56 issues have been closed, including 5 fixed issues on the core;
- 55 pull requests have been opened in the project repositories;
- 74 pull requests have been closed, including 51 merged pull requests.
Code changes in the ‘develop’ branch
Core
- #16433: Update license headers, by @matks
- #16331: Add “total_shipping_tax_excl” and “total_shipping_tax_incl” in email data. Thank you @yannicka
- #16316: ImageManager::validateUpload : MimeType can’t be check for SVG files, by @Progi1984
- #16310: Add ps_dataprivacy, productcomments and ps_crossselling as native modules, by @PierreRambaud
- #16213: Make id order available. Thank you @atomiix
- #15902: Use the correct environment depending on what you are doing (dev, test, and prod), by @PierreRambaud
Back office
- #16394: Fix typo in condition. Thank you @mvorisek
- #16392: Fixed the change the quantity of a free product in an order in the BO, by @Progi1984
- #16375: Emails are not case sensitive, by @PierreRambaud
- #16360: Twig\HookExtension: Bugfix ob level in renderHooksArray. Thank you @jnvsor
- #15915: Migration of International > Taxes > Tax rules page, by @matks
- #15229: Migration of Sell > Customers > Addresses page. Thank you @RaimondasSapola
- #14972: Add/Edit a currency with the CLDR implementation, by @jolelievre
- #14288: Migrate Customer Service view. Thank you @sarjon
- #13716: Migrate Sell > Catalog > Catalog price rule create/edit action. Thank you @zuk3975
Tests
- #16414: Tests - Replace duplicated steps with foreach in taxes tests, by @boubkerbribri
- #16408: Tests - Replace duplicated steps with foreach in customers tests, by @boubkerbribri
- #16407: Tests - Replace duplicated steps with foreach in categories tests, by @boubkerbribri
- #16405: Tests - Replace duplicated steps with foreach in suppliers tests, by @boubkerbribri
- #16403: Tests - Replace duplicated steps with foreach for brands tests, by @boubkerbribri
- #16399: Tests - Delete replaces in tests Taxes, by @boubkerbribri
- #16395: Tests - Delete replaces in tests Customers, by @boubkerbribri
- #16387: Tests - Delete replaces in tests categories, by @boubkerbribri
- #16373: Tests - Using employees demo file for installation, by @boubkerbribri
- #16371: Tests - Delete replaces and use new methods instead in Supplier tests, by @boubkerbribri
- #16332: Tests - using same demo elements, by @boubkerbribri
- #16330: Tests - Delete replaces in tests and use new methods instead, by @boubkerbribri
- #16317: Only trigger deploy on cron task, and tests on push / pull requests, by @PierreRambaud
Code changes in the ‘1.7.6.x’ branch
Core
Back office
- #16365: Viewed products - the date is showing product id - not a date, by @khouloudbelguith
Front office
- #16423: Use default attribute when no one are passed in front page, by @PierreRambaud
Code changes in modules, themes & tools
ps_currencyselector
- #11: Rebuild the url to avoid doublons, by @jolelievre
QA nightly results
- #10: Add jquery locally, change version system, by @SimonGrn
- #9: Inverted sorting, by @SimonGrn
- #8: Various fix, by @SimonGrn
Product Comments module
- #44: Update AFL-3.0 identifier, by @jolelievre
- #43: Update AFL-3.0 identifier, by @jolelievre
- #41: Merge dev to master for v4.0.1, by @jolelievre
Changes in developer documentation
- #403: Update afl license, by @Quetzacoalt91
- #402: Provide module and context properties in ModuleFrontController, by @Quetzacoalt91
- #400: Add documentation for row actions, and clickable_row option, by @jolelievre
- #396: Improve style of notice blocks, by @eternoendless
blockreassurance
- #36: Added a template for PullRequest, by @Progi1984
- #35: Permit to upload a SVG file as custom icon, by @Progi1984
- #34: Migrated JS & CSS assets from files to NPM & Webpack, by @Progi1984
- #33: Changed displayed images in second pack of images, by @Progi1984
Cronjobs module
- #49: Add missing domains, by @LouiseBonnard
Email Alerts module
- #25: Add missing domains, by @LouiseBonnard
Gamification module
- #70: Add missing domains, by @LouiseBonnard
dateofdelivery
- #18: Add missing domains, by @LouiseBonnard
Thank you to the contributors whose pull requests were merged since the last Core Weekly Report: @matks, @PierreRambaud, @boubkerbribri, @jolelievre, @mvorisek, @Progi1984, @SimonGrn, @khouloudbelguith, @jnvsor, @Quetzacoalt91, @atomiix, @yannicka, @eternoendless, @LouiseBonnard, @RaimondasSapola, @sarjon, @zuk3975!
Thank you to the contributors whose PRs haven’t been merged yet! And of course, a big thank you to all those who contribute with issues and comments on GitHub!
If you want to contribute to PrestaShop with code, please read these pages first:
…and if you do not know how to fix an issue but wish to report it, please read this: How to use GitHub to report an issue. Thank you!
Happy contributin’ everyone!