User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Currently, the method getForm of backend ArticleModel has code which handle form fields adjustment form both frontend and backend (thus in some places, it has to use $app->isClient to check to see if it is dealing with frontend or backend submit article, it even has to use application input to get article id..), so it is quite messy.
I re-arrange the code of that model. Instead of put everything into backend ArticleModel model, I move the code (which is related to frontend submit article only) to frontend FormModel to make it more clean and easier to maintain.
I also made some small adjustment:
|Category||⇒||Administration com_content Front End|
I would say no. In this PR, I just try to improve the code to make it more clean and easier to maintainer. However, there might be merge conflict.
I checked: no merge conflicts, at least if your patch is merged first.
With the nightly of 20 feb:
Notice: Undefined index: catid in /var/www/vhosts/jugmi.it/subdomains/j4/httpdocs/administrator/components/com_content/src/Model/ArticleModel.php on line 502
|Status||Pending||⇒||Ready to Commit|
|Status||Ready to Commit||⇒||Fixed in Code Base|
|Closed_Date||0000-00-00 00:00:00||⇒||2021-02-20 17:42:08|