[#32466] - [4.0] Improve/Restructure Article Model
- Fixed in Code Base
- 20 Feb 2021
- Medium
- Build: 4.0-dev
- # 32466
- Diff
- joomdonation:submit_article_improvement
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
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:
- Fix the wrong type casting here https://github.com/joomla/joomla-cms/blob/4.0-dev/administrator/components/com_content/src/Model/ArticleModel.php#L529 (when $form->getValue return array, it is casted to int with wrong value)
- Remove un-used JLoader::register here https://github.com/joomla/joomla-cms/blob/4.0-dev/administrator/components/com_content/src/Model/ArticleModel.php#L167 because we are using namespace class already
- Made some code style fix.
Testing Instructions
- Apply patch
- Create/edit article from backend, make sure it is working as expected
- Create a menu item to link to Create Article menu option of com_content, submit an article, make sure it works.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | 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 hate merge conflicts... ;)
Allow me to improve the existing bad code first :).
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
| Labels |
Added:
?
|
||
Sorry @Razzo1987, there was a small typo in the code. I fixed it. Could you please help testing it again?
I have tested this item
Tested from backend and frontend with success
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32466.
I have tested this item
done
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32466.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32466.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-02-20 17:42:08 |
| Closed_By | ⇒ | infograf768 | |
| Labels |
Added:
?
|
||
Tks.
Would that have an impact on #32462 ?