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
0001108 [In-Portal CMS] Admin Interfaces bug report always 2011-08-24 03:41 2012-07-25 05:32
Reporter phil View Status public Project Name In-Portal CMS
Assigned To alex Developer
Priority normal Resolution fixed Fixed in Version 5.2.0-B1
Status closed Product Version 5.1.3-RC2 Target Version 5.2.0
Time EstimateNo estimate
Summary 0001108: Selecting "Symlink Category" resets all category adding form fields
Description When creating a new category, if you try to setup a symlink, page reloads with all fields emptied. If you create the page, save it, then edit it and setup a symlink, it works.

Way of fixing:
Add ability to submit category(-ies)/category item(-s) selected in item selector to parent window using ajax
Additional Information
Tags No tags attached.
Reference https://groups.google.com/d/topic/in-portal-bugs/U7tgCKfPEqI/discussion
Change Log Message Fixed all data lost of symlink category selection on category adding form
Estimate Points 1
Attached Files patch file icon symlink_category_1108.patch [^] (6,837 bytes) 2011-12-19 13:17 [Show Content]
patch file icon symlink_category_1108.2.patch [^] (3,703 bytes) 2011-12-21 06:44 [Show Content]
? file icon category_path.tpl [^] (342 bytes) 2011-12-23 11:42
patch file icon symlink_category_v3.patch [^] (5,774 bytes) 2011-12-27 04:45 [Show Content]
png file icon SymlinkJSError.png [^] (232,487 bytes) 2011-12-27 10:03


patch file icon unit_prefix_bug_fix.patch [^] (1,018 bytes) 2011-12-28 04:59 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
User avatar (0004265)
Dmitry (manager)
2011-12-15 17:27

Consult with Alex on the implementation of this task.
User avatar (0004278)
erik (manager)
2011-12-19 10:14

Patch attached - needs testing
User avatar (0004280)
alex (manager)
2011-12-20 03:33

Works, but implementation is wrong:

1. should be done, like with user selector:
- check if processItemSelector function exists in a parent window
- if so, then call it passing dst_field & selector window
- create processItemSelector function in "inp_edit_category" block, that will do ajax query (using jQuery) to a TPL that would return category path
- just include intermediate template in "inp_edit_category" block instead of making ajax query on first page load (bad style), since jumping effect is visible otherwise
- can use intermediate event (if needed) to transform "selected_ids" into a form, that is understandable by CategoryPath tag, you've used
User avatar (0004291)
erik (manager)
2011-12-21 06:45

Made changes in implementation. New patch symlink_category_1108.2.patch attached - needs testing.
User avatar (0004302)
alex (manager)
2011-12-23 04:43

File "category_path.tpl" is missing in 2nd patch and I don't if it was changed since first patch. Please add missing files.

Please verify that all needed changes are in the patch after creating it and before sending it to testing.
User avatar (0004310)
erik (manager)
2011-12-23 11:43

category_path.tpl was not changed. File attached - needs testing
User avatar (0004319)
alex (manager)
2011-12-27 04:46
edited on: 2011-12-27 04:47

Works, but technically method CategorySelector.getField could have been used to get desired $field_id instead of duplicating it's code in processItemSelector method.

Also passing parameter directly to "category_path" template is more preferable, then setting global variable.

User avatar (0004320)
alex (manager)
2011-12-27 04:47

Fix committed to 5.2.x branch. Commit Message:

Fixes 0001108: Selecting "Symlink Category" resets all category adding form fields
User avatar (0004328)
Dmitry (manager)
2011-12-27 10:02

There is a JS error when trying to select a Section in Show All mode (see attached file)
User avatar (0004331)
alex (manager)
2011-12-28 02:40

Before this patch event "c:OnProcessSelected" was called. Within it "selected_ids" represented IDs of category, that was selected.

If you look closed on the JS code, that called "c:OnProcessSelected" event (which you changed), then you'll notice, that actual IDs were retrieved using Grids[$Catalog.ActivePrefix].GetSelected() method.

This way instead of specifying "c" directly (as unit prefix) you will use $opener.$Catalog.ActivePrefix instead.
User avatar (0004332)
erik (manager)
2011-12-28 05:00

unit_prefix_bug_fix.patch attached - needs testing
User avatar (0004340)
alex (manager)
2011-12-28 12:10

Fix committed to 5.2.x branch. Commit Message:

Fixes 0001108: Selecting "Symlink Category" resets all category adding form fields
1. "Show All" mode generated JavaScript error
User avatar (0005016)
alex (manager)
2012-07-25 05:32

Since 5.2.0 version was released.

- Related Changesets
In-Portal CMS: 5.2.x r14936
Timestamp: 2011-12-28 12:10:00
Author: alex
Details ] Diff ]
Fixes 0001108: Selecting "Symlink Category" resets all category adding form fields
1. "Show All" mode generated JavaScript error
mod - /in-portal/branches/5.2.x/core/admin_templates/incs/form_blocks.tpl Diff ] File ]
In-Portal CMS: 5.2.x r14916
Timestamp: 2011-12-27 04:47:26
Author: alex
Details ] Diff ]
Fixes 0001108: Selecting "Symlink Category" resets all category adding form fields
mod - /in-portal/branches/5.2.x/core/admin_templates/catalog/item_selector/item_selector_toolbar.tpl Diff ] File ]
add - /in-portal/branches/5.2.x/core/admin_templates/categories/category_path.tpl File ]
mod - /in-portal/branches/5.2.x/core/admin_templates/incs/form_blocks.tpl Diff ] File ]

- Issue History
Date Modified Username Field Change
2012-07-25 05:32 alex Note Added: 0005016
2012-07-25 05:32 alex Status resolved => closed
2011-12-28 12:10 alex Note Added: 0004340
2011-12-28 12:10 alex Status reviewed and tested => resolved
2011-12-28 12:10 alex Resolution reopened => fixed
2011-12-28 12:10 alex Assigned To !COMMUNITY => alex
2011-12-28 12:10 alex Changeset attached 5.2.x r14936
2011-12-28 12:09 alex Assigned To alex => !COMMUNITY
2011-12-28 12:09 alex Status needs testing => reviewed and tested
2011-12-28 05:00 erik Note Added: 0004332
2011-12-28 05:00 erik Assigned To erik => alex
2011-12-28 05:00 erik Status needs work => needs testing
2011-12-28 04:59 erik File Added: unit_prefix_bug_fix.patch
2011-12-28 02:40 alex Note Added: 0004331
2011-12-27 10:03 Dmitry File Added: SymlinkJSError.png
2011-12-27 10:02 Dmitry Status needs feedback => needs work
2011-12-27 10:02 Dmitry Assigned To alex => erik
2011-12-27 10:02 Dmitry Note Added: 0004328
2011-12-27 10:02 Dmitry Status resolved => needs feedback
2011-12-27 10:02 Dmitry Resolution fixed => reopened
2011-12-27 04:47 alex Note Added: 0004320
2011-12-27 04:47 alex Status reviewed and tested => resolved
2011-12-27 04:47 alex Fixed in Version => 5.2.0-B1
2011-12-27 04:47 alex Resolution open => fixed
2011-12-27 04:47 alex Assigned To !COMMUNITY => alex
2011-12-27 04:47 alex Changeset attached 5.2.x r14916
2011-12-27 04:47 alex Note Edited: 0004319 View Revisions
2011-12-27 04:46 alex Note Added: 0004319
2011-12-27 04:46 alex Assigned To alex => !COMMUNITY
2011-12-27 04:46 alex Status needs testing => reviewed and tested
2011-12-27 04:45 alex File Added: symlink_category_v3.patch
2011-12-23 11:43 erik Note Added: 0004310
2011-12-23 11:43 erik Assigned To erik => alex
2011-12-23 11:43 erik Status needs work => needs testing
2011-12-23 11:42 erik File Added: category_path.tpl
2011-12-23 04:43 alex Note Added: 0004302
2011-12-23 04:43 alex Assigned To alex => erik
2011-12-23 04:43 alex Status needs testing => needs work
2011-12-21 06:45 erik Note Added: 0004291
2011-12-21 06:45 erik Assigned To erik => alex
2011-12-21 06:45 erik Status needs work => needs testing
2011-12-21 06:44 erik File Added: symlink_category_1108.2.patch
2011-12-20 03:33 alex Note Added: 0004280
2011-12-20 03:33 alex Assigned To alex => erik
2011-12-20 03:33 alex Status needs testing => needs work
2011-12-19 13:17 erik File Added: symlink_category_1108.patch
2011-12-19 13:17 erik File Deleted: symlink_category_1108.patch
2011-12-19 10:14 erik Note Added: 0004278
2011-12-19 10:14 erik Assigned To erik => alex
2011-12-19 10:14 erik Developer => erik
2011-12-19 10:14 erik Status needs work => needs testing
2011-12-19 10:13 erik File Added: symlink_category_1108.patch
2011-12-18 14:59 alex Estimate Points 0 => 1
2011-12-15 17:27 Dmitry Estimate Points => 0
2011-12-15 17:27 Dmitry Note Added: 0004265
2011-12-15 17:27 Dmitry Assigned To => erik
2011-12-15 17:27 Dmitry Status active => needs work
2011-12-15 17:27 Dmitry Target Version Icebox => 5.2.0
2011-08-24 03:41 alex Reporter alex => phil
2011-08-24 03:41 alex New Issue
2011-08-24 03:41 alex Reference => https://groups.google.com/d/topic/in-portal-bugs/U7tgCKfPEqI/discussion
2011-08-24 03:41 alex Change Log Message => Fixed all data lost of symlink category selection on category adding form



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

Powered by Mantis Bugtracker