Ng Kiat Siong

Results 26 issues of 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...

Component: Customer
Template : rwd
Template : base

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.~ ![image](https://user-images.githubusercontent.com/1106470/193274575-a9bcfadd-ec40-456c-bf3f-88506d8d1de5.png) ### Fixed Issues (if relevant)...

Component: Core
translations

### 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...

Component: Core
Component: Catalog
Component: Sales
Component: Customer
Component: Adminhtml
Component: CurrencySymbol
Component: Admin
Component: SalesRule
Component: Weee
Component: CatalogIndex
Component: CatalogSearch
Component: Dataflow
Component: Install

…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...

performance
Component: Sales

### 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...

Component: Eav

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,...

Component: Api2

Minor docblock fixes.

Mage.php

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.

Component: PayPal
Component: Adminhtml
Mage.php

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...

Component: Customer
Component: Adminhtml
translations