Anonymous | Login | Signup for a new account | 2024-03-29 01:42 CDT |
Main | My View | View Issues | Change Log | Roadmap | Docs | Wiki | Repositories |
Viewing Issue Simple Details [ Jump to Notes ] [ Wiki ] | [ View Advanced ] [ Issue History ] [ Print ] | ||||||
ID | Category | Type | Reproducibility | Date Submitted | Last Update | ||
0000661 | [In-Portal CMS] Front End | bug report | always | 2010-03-27 17:25 | 2011-03-30 09:58 | ||
Reporter | alex | View Status | public | Project Name | In-Commerce | ||
Assigned To | alex | Developer | |||||
Priority | normal | Resolution | fixed | Fixed in Version | 5.1.1-B1 | ||
Status | closed | Product Version | 5.0.3-B1 | Target Version | 5.1.1 | ||
Time Estimate | No estimate | ||||||
Summary | 0000661: Possible no permission error, while accessing shipping type details on Front-End | ||||||
Description |
It's possible, that no permission error will occur, while accessing shipping type details on Front-End. Problem source could be hidden inside ShippingEventHandler::OnAfterItemLoad method contents. Inside that method object of "sc" prefix (shipping cost) is retrieved and manipulated. It's not clear to me why we need to do that all, but still need to checked out. To test that you need to create a custom tag (in ShippingTagProcessor), with such code: $object =& $this->getObject( Array ('skip_autoload' => true) ); /* @var $object kDBItem */ $object->Load(1); Shipping type with ID = 1 already should exist before testing is started. Perform this on Front-End with logged-in/guest users and in administrative console (on shipping type editing) by "root" / admin user. |
||||||
Additional Information | |||||||
Tags | No tags attached. | ||||||
Reference | http://groups.google.com/group/in-portal-bugs/browse_thread/thread/a3889ac5acf1afbf | ||||||
Change Log Message | |||||||
Estimate Points | 0 | ||||||
Attached Files | shipping_cost_warning_fix.patch [^] (891 bytes) 2010-09-08 11:52 [Show Content] | ||||||
|
Relationships [ Relation Graph ] [ Dependency Graph ] | |
Notes | |
(0002791) alex (manager) 2010-09-08 11:54 |
That code under question should only be executed in administrative console. That's what the patch fixes. |
(0002795) Dmitry (manager) 2010-09-08 23:30 |
Tested, please commit. |
(0002802) alex (manager) 2010-09-10 02:57 |
Fix committed to 5.1.x branch. Commit Message: Fixes 0000661: Possible no permission error, while accessing shipping type details on Front-End |
(0003391) Dmitry (manager) 2011-03-30 09:58 |
Closing, since 5.1.1 release has been released. |
Related Changesets | |||
Modules :: In-Commerce: 5.1.x r13928 Timestamp: 2010-09-10 02:57:30 Author: alex [ Details ] [ Diff ] |
Fixes 0000661: Possible no permission error, while accessing shipping type details on Front-End | ||
mod - /w/in-commerce/branches/5.1.x/units/shipping/shipping_event_handler.php | [ Diff ] [ File ] | ||
Issue History | |||
Date Modified | Username | Field | Change |
2011-03-30 09:58 | Dmitry | Note Added: 0003391 | |
2011-03-30 09:58 | Dmitry | Status | resolved => closed |
2010-09-10 02:57 | alex | Changeset attached | 5.1.x r13928 |
2010-09-10 02:57 | alex | Note Added: 0002802 | |
2010-09-10 02:57 | alex | Status | reviewed and tested => resolved |
2010-09-10 02:57 | alex | Fixed in Version | => 5.1.1-B1 |
2010-09-10 02:57 | alex | Resolution | open => fixed |
2010-09-08 23:30 | Dmitry | Note Added: 0002795 | |
2010-09-08 23:30 | Dmitry | Assigned To | Dmitry => alex |
2010-09-08 23:30 | Dmitry | Status | needs testing => reviewed and tested |
2010-09-08 11:54 | alex | Time Estimate Removed | 1 => |
2010-09-08 11:54 | alex | Note Added: 0002791 | |
2010-09-08 11:54 | alex | Assigned To | alex => Dmitry |
2010-09-08 11:54 | alex | Developer | => alex |
2010-09-08 11:54 | alex | Status | needs work => needs testing |
2010-09-08 11:52 | alex | File Added: shipping_cost_warning_fix.patch | |
2010-08-27 13:16 | alex | Time Estimate Added | 1 |
2010-07-31 13:31 | Dmitry | Assigned To | => alex |
2010-07-31 13:31 | Dmitry | Status | active => needs work |
2010-07-31 13:31 | Dmitry | Target Version | Icebox => 5.1.1 |
2010-05-10 14:51 | alex | Target Version | 5.0.4 => Icebox |
2010-03-27 17:25 | alex | New Issue | |
2010-03-27 17:25 | alex | Reference | => http://groups.google.com/group/in-portal-bugs/browse_thread/thread/a3889ac5acf1afbf |
Main | My View | View Issues | Change Log | Roadmap | Docs | Wiki | Repositories |
Web Development by Intechnic In-Portal Open Source CMS |