In-Portal Issue Tracker

Welcome to the In-Portal Open Source CMS Issue Tracker! This is a central management / tracking tool for all types of tasks / issues / bugs for the In-Portal Project. Before reporting any issues, please make sure to read the Guide into Issue Tracker and How to Properly Test and Report Bugs!

Viewing Issue Simple Details Jump to Notes ] Wiki ]  Related Changesets ] View Advanced ] Issue History ] Print ]
ID Category Type Reproducibility Date Submitted Last Update
0000357 [In-Portal CMS] Data Management bug report always 2009-10-06 05:48 2010-07-22 15:07
Reporter alex View Status public Project Name In-Portal CMS
Assigned To alex Developer
Priority normal Resolution fixed Fixed in Version 5.1.0
Status closed Product Version 5.0.1 Target Version 5.1.0
Time EstimateNo estimate
Summary 0000357: Form modification detector (it's php part) works strange
Description When I open link (in catalog) for editing and click on "General" tab without changing anything, then message "Modifications will not take effect until you click the Save button!" is shown, but I didn't change a thing.

It is possible, that something I don't see on form was changed, but then such fields shouldn't cause such warning message to appear.

It's possible, that other forms also have same behavior.
Additional Information We have field named "Modified" in certain tables (including category items), that holds last modification date of and database record. There are several problems in current implementation:

    * field is updated no matter was something changed or not;
    * field is always updated no matter what user have entered on editing template.

I propose to fix that by:

    * check that other fields (except that Modified) field were changed before changing it
    * make Modified field on editing form as label, since user entered value will be always ignored.

Same about ModifiedById field around the system.
Tags No tags attached.
Reference http://groups.google.com/group/in-portal-bugs/browse_thread/thread/5fe6f07765659682
Change Log Message
Estimate Points 0
Attached Files patch file icon form_modification_detector_part1.patch [^] (2,848 bytes) 2010-03-02 08:36 [Show Content]
patch file icon modification_detector_core.patch [^] (3,851 bytes) 2010-05-26 04:16 [Show Content]
patch file icon modification_detector_modules.patch [^] (7,357 bytes) 2010-05-26 04:16 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
related to 0000580closed (5.1.1)Dmitry In-Portal CMS Issues with change log section 
parent of 0000910closed (5.1.1)alex In-Link Fatal error when Modifying Pending Link on Front-end 
related to 0000581closed (5.0.4)alex In-Commerce Wrong calculation of HasRequiredOptions field resulting additional change log record 
child of 0000107closed (5.1.0)alex In-Portal CMS Implement "MemCached" functionality 

-  Notes
User avatar (0001476)
alex (manager)
2010-02-11 14:33

Moving to 5.1.0, because this will be implemented to use in memcache functionality.
User avatar (0001487)
alex (manager)
2010-02-24 11:58

After all this task wasn't involved in memcache task, so move it back.
User avatar (0001502)
alex (manager)
2010-03-02 08:37

Patch "form_modification_detector_part1.patch" is for categories and category items only. All other unit configs should be also scanned to complete this task.
User avatar (0002323)
alex (manager)
2010-05-26 04:17

Latest 2 patches are ones to be tested.

Also apply patch from 0000581 task before starting testing.
User avatar (0002341)
Dmitry (manager)
2010-05-30 22:28
edited on: 2010-05-30 22:28

Tested good. Please commit to 5.1.0.

I believe no reason to push this back since 5.2.0 will be converting to PHP5.

User avatar (0002485)
alex (manager)
2010-07-07 04:41

Fix committed to 5.1.x branch. Commit Message:

Fixes 0000357: Form modification detector (it's php part) works strange
User avatar (0002577)
alex (manager)
2010-07-22 15:07

Closing, since 5.1.0 release has been released.

- Related Changesets
In-Portal CMS: 5.1.x r13811
Timestamp: 2010-07-07 04:41:05
Author: alex
Details ] Diff ]
Fixes 0000357: Form modification detector (it's php part) works strange
mod - /in-portal/branches/5.1.x/core/install/upgrades.sql Diff ] File ]
mod - /in-portal/branches/5.1.x/core/kernel/db/cat_dbitem.php Diff ] File ]
mod - /in-portal/branches/5.1.x/core/units/categories/categories_event_handler.php Diff ] File ]
mod - /in-portal/branches/5.1.x/core/units/helpers/email_message_helper.php Diff ] File ]
Modules :: In-News: 5.1.x r13810
Timestamp: 2010-07-07 04:38:10
Author: alex
Details ] Diff ]
Bug 0000357: Form modification detector (it's php part) works strange
mod - /modules/in-news/branches/5.1.x/admin_templates/category_properties.tpl Diff ] File ]
Modules :: In-Commerce: 5.1.x r13809
Timestamp: 2010-07-07 04:37:47
Author: alex
Details ] Diff ]
Bug 0000357: Form modification detector (it's php part) works strange
mod - /w/in-commerce/branches/5.1.x/admin_templates/products/products_access.tpl Diff ] File ]
mod - /w/in-commerce/branches/5.1.x/admin_templates/products/products_files.tpl Diff ] File ]
mod - /w/in-commerce/branches/5.1.x/install/install_schema.sql Diff ] File ]
mod - /w/in-commerce/branches/5.1.x/install/upgrades.sql Diff ] File ]
mod - /w/in-commerce/branches/5.1.x/units/pricing/pricing_event_handler.php Diff ] File ]
mod - /w/in-commerce/branches/5.1.x/units/products/products_config.php Diff ] File ]

- Issue History
Date Modified Username Field Change
2010-10-31 08:19 alex Relationship added parent of 0000910
2010-07-22 15:07 alex Note Added: 0002577
2010-07-22 15:07 alex Status resolved => closed
2010-07-07 04:41 alex Note Added: 0002485
2010-07-07 04:41 alex Status reviewed and tested => resolved
2010-07-07 04:41 alex Fixed in Version => 5.1.0
2010-07-07 04:41 alex Resolution open => fixed
2010-07-07 04:41 alex Changeset attached 5.1.x r13811
2010-07-07 04:38 alex Changeset attached 5.1.x r13810
2010-07-07 04:37 alex Changeset attached 5.1.x r13809
2010-07-07 03:57 alex Target Version Icebox => 5.1.0
2010-05-30 22:28 Dmitry Note Edited: 0002341 View Revisions
2010-05-30 22:28 Dmitry Note Added: 0002341
2010-05-30 22:28 Dmitry Assigned To !COMMUNITY => alex
2010-05-30 22:28 Dmitry Status needs testing => reviewed and tested
2010-05-26 04:17 alex Note Added: 0002323
2010-05-26 04:17 alex Assigned To => !COMMUNITY
2010-05-26 04:17 alex Developer => alex
2010-05-26 04:17 alex Status active => needs testing
2010-05-26 04:16 alex File Added: modification_detector_modules.patch
2010-05-26 04:16 alex File Added: modification_detector_core.patch
2010-05-26 03:50 alex Relationship added related to 0000581
2010-05-10 14:50 alex Target Version 5.0.4 => Icebox
2010-05-10 14:33 alex Relationship added related to 0000580
2010-03-14 14:17 alex Reference => http://groups.google.com/group/in-portal-bugs/browse_thread/thread/5fe6f07765659682
2010-03-14 14:16 alex Additional Information Updated View Revisions
2010-03-02 08:37 alex Note Added: 0001502
2010-03-02 08:36 alex File Added: form_modification_detector_part1.patch
2010-02-24 11:58 alex Note Added: 0001487
2010-02-24 11:58 alex Target Version 5.1.0 => 5.0.4
2010-02-11 14:34 alex Relationship added child of 0000107
2010-02-11 14:33 alex Note Added: 0001476
2010-02-11 14:33 alex Target Version 5.0.3 => 5.1.0
2010-01-06 06:25 alex Target Version Icebox => 5.0.3
2009-10-06 05:48 alex Target Version => Icebox
2009-10-06 05:48 alex New Issue



Web Development by Intechnic
In-Portal Open Source CMS
In-Portal Open Source CMS
Copyright © 2000 - 2009 MantisBT Group

Powered by Mantis Bugtracker