Ng Kiat Siong
Ng Kiat Siong
### Description (*) The mini login form should only be used on SSL enabled pages. But nowadays, the entire production site should be SSL enabled. Mini login form makes it...
Is it a good idea? I recently made a cron job to insert millions of rows from another database. I need to know the progress of the job and at...
### Description (*) Config `general/reprocess_images/active` is removed and replaced with `admin/security/reprocess_image_quality`, where it is default to 85. ~For BC, checking of `general/reprocess_images/active` takes precedent.~  ### Fixed Issues (if relevant)...
### Description (*) These conditions ```php isset($someVar[0]) && !empty($someVar[0]; isset($someVar[0]) && $someVar[0]; ``` are the same as ```php !empty($someVar[0]); ``` More info [here](https://phpdelusions.net/articles/empty). ### Questions or comments To me, this...
…r is saved. ### Description (*) Order is saved when comments are added in the backend. Before this PR, this will save the new comment (status history object) along with...
### Description (*) This PR addresses 2 issues. #### Custom `source_model` When adding a new attribute in which the input type is select, the source model is always set to...
With this PR, it is now possible to response with 201 HTTP_CREATE on REST POST. [edit] The following no longer applied for this PR. ### Description (*) In REST POST,...
It is referenced in [23 files](https://github.com/search?q=repo%3AOpenMage%2Fmagento-lts+Mage::getBlockSingleton&type=code). The comment stated that it should be replaced by `Mage::helper()`, but that is a totally different function.
Trying to rebase PR #1283 to the `main` branch. ### Fixed Issues (if relevant) 1. Fixes #1205 ### Description (*) When a customer account is created in backed, the password...