Anonymous | Login | Signup for a new account | 2024-04-20 04:26 CDT |
Main | My View | View Issues | Change Log | Roadmap | Docs | Wiki | Repositories |
Viewing Issue Advanced Details [ Jump to Notes ] [ Wiki ] [ Related Changesets ] | [ View Simple ] [ Issue History ] [ Print ] | ||||||
ID | Category | Type | Reproducibility | Date Submitted | Last Update | ||
0001355 | [In-Portal CMS] Database | feature request | N/A | 2012-07-16 11:37 | 2012-07-25 05:29 | ||
Reporter | alex | View Status | public | Project Name | In-Portal CMS | ||
Assigned To | alex | Developer | |||||
Priority | normal | Resolution | fixed | Platform | |||
Status | closed | OS | |||||
OS Version | |||||||
ETA | none | Fixed in Version | 5.2.0 | Product Version | 5.2.0-RC1 | ||
Target Version | 5.2.0 | Product Build | |||||
Time Estimate | No estimate | ||||||
Summary | 0001355: Ability to update only given fields via kDBItem::Update method | ||||||
Description |
Right now, when some fields of kDBItem object are changed and kDBItem::Update method is issued then all fields are put into UPDATE sql and not only ones, that were changed. I'm proposing to add 2nd optional parameter (1st parameter is $id) to kDBItem::Update method to allow specifying fields, that should only be updated in database. Also only these fields will be overwritten in kDBItem::OriginalFieldValues array. This way there is still way to detect change in field values, that were not specified to kDBItem::Update method. |
||||||
Steps To Reproduce | |||||||
Additional Information | |||||||
Tags | No tags attached. | ||||||
Reference | https://groups.google.com/d/topic/in-portal-dev/3U8_1WTddIg/discussion | ||||||
Change Log Message | Adds ability to update specific db fields only | ||||||
Estimate Points | 1 | ||||||
Attached Files |
specific_field_db_update_core.patch [^] (3,688 bytes) 2012-07-16 11:37
[Show Content]
specific_field_db_update_modules.patch [^] (1,172 bytes) 2012-07-16 11:37 [Show Content] specific_field_db_update_core_513.patch [^] (3,441 bytes) 2012-07-16 12:09 [Show Content] specific_field_db_update_modules_513.patch [^] (1,154 bytes) 2012-07-16 12:10 [Show Content] specific_field_db_update_core_addon_513.patch [^] (555 bytes) 2012-07-18 04:19 [Show Content] |
||||||
|
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||
|
Notes | |
(0004788) alex (manager) 2012-07-16 11:41 |
Will test all together later. |
(0004789) alex (manager) 2012-07-16 11:42 |
Fix committed to 5.2.x branch. Commit Message: Fixes 0001355: Ability to update only given fields via kDBItem::Update method |
(0004801) alex (manager) 2012-07-18 04:20 |
Some code in 5.2.0 was reorganized a way, that it doesn't have equivalent in 5.1.3 and other releases before it. Because of this I wasn't able to properly port 5.2.0 patch changes to 5.1.3 and forget 1 place in "CategoriesItem" class. This has been fixed by "specific_field_db_update_core_addon_513.patch" patch. |
(0004829) alex (manager) 2012-07-25 05:29 |
Since 5.2.0 version was released. |
Related Changesets | |||
In-Portal CMS: 5.2.x r15454 Timestamp: 2012-07-16 11:42:50 Author: alex [ Details ] [ Diff ] |
Fixes 0001355: Ability to update only given fields via kDBItem::Update method | ||
mod - /in-portal/branches/5.2.x/core/kernel/db/cat_dbitem.php | [ Diff ] [ File ] | ||
mod - /in-portal/branches/5.2.x/core/kernel/db/dbitem.php | [ Diff ] [ File ] | ||
mod - /in-portal/branches/5.2.x/core/units/images/image_event_handler.php | [ Diff ] [ File ] | ||
mod - /in-portal/branches/5.2.x/core/units/users/users_item.php | [ Diff ] [ File ] | ||
Modules :: In-Commerce: 5.2.x r15453 Timestamp: 2012-07-16 11:41:43 Author: alex [ Details ] [ Diff ] |
Bug 0001355: Ability to update only given fields via kDBItem::Update method | ||
mod - /w/in-commerce/branches/5.2.x/units/shipping_costs/shipping_costs_event_handler.php | [ Diff ] [ File ] | ||
Issue History | |||
Date Modified | Username | Field | Change |
2012-07-25 05:29 | alex | Note Added: 0004829 | |
2012-07-25 05:29 | alex | Status | resolved => closed |
2012-07-18 04:20 | alex | Note Added: 0004801 | |
2012-07-18 04:19 | alex | File Added: specific_field_db_update_core_addon_513.patch | |
2012-07-16 12:10 | alex | File Added: specific_field_db_update_modules_513.patch | |
2012-07-16 12:09 | alex | File Added: specific_field_db_update_core_513.patch | |
2012-07-16 11:42 | alex | Changeset attached | 5.2.x r15454 |
2012-07-16 11:42 | alex | Note Added: 0004789 | |
2012-07-16 11:42 | alex | Status | reviewed and tested => resolved |
2012-07-16 11:42 | alex | Fixed in Version | => 5.2.0 |
2012-07-16 11:42 | alex | Resolution | open => fixed |
2012-07-16 11:42 | alex | Assigned To | !COMMUNITY => alex |
2012-07-16 11:41 | alex | Changeset attached | 5.2.x r15453 |
2012-07-16 11:41 | alex | Note Added: 0004788 | |
2012-07-16 11:41 | alex | Status | needs testing => reviewed and tested |
2012-07-16 11:41 | alex | Assigned To | => !COMMUNITY |
2012-07-16 11:41 | alex | Developer | => alex |
2012-07-16 11:41 | alex | Status | active => needs testing |
2012-07-16 11:40 | alex | Reference | => https://groups.google.com/d/topic/in-portal-dev/3U8_1WTddIg/discussion |
2012-07-16 11:38 | alex | Relationship added | related to 0000043 |
2012-07-16 11:38 | alex | Relationship deleted | related to 0000835 |
2012-07-16 11:37 | alex | Relationship added | related to 0000835 |
2012-07-16 11:37 | alex | File Added: specific_field_db_update_modules.patch | |
2012-07-16 11:37 | alex | File Added: specific_field_db_update_core.patch | |
2012-07-16 11:37 | alex | New Issue | |
2012-07-16 11:37 | alex | Change Log Message | => Adds ability to update specific db fields only |
2012-07-16 11:37 | alex | Estimate Points | => 1 |
Main | My View | View Issues | Change Log | Roadmap | Docs | Wiki | Repositories |
Web Development by Intechnic In-Portal Open Source CMS |