Changeset 14436

Show
Ignore:
Timestamp:
01/15/08 20:04:18 (11 months ago)
Author:
flack
Message:

make selection work more similar in both allow_multiple modes

(this makes it possible to empty a field and delete the old value)

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/midcom/midcom.helper.datamanager2/static/chooser/jquery.chooser_widget.js

    r14435 r14436  
    514514            var current_presel_status = jQuery(li_element).attr("pre_selected"); 
    515515             
    516             // On allow_multiple create multiple items that can be selected at the same moment 
    517             if (options.allow_multiple) 
     516            if (current_keep_status == "true") 
    518517            { 
    519                 if (current_keep_status == "true") 
    520                 { 
    521                     if (current_delete_status == "false") 
     518                if (current_delete_status == "false") 
     519                { 
     520                    jQuery(li_element).removeClass(CLASSES.ACTIVE); 
     521                    if (current_presel_status == "true") 
    522522                    { 
    523                         jQuery(li_element).removeClass(CLASSES.ACTIVE); 
    524                         if (current_presel_status == "true") 
    525                         { 
    526                             remove(item_id); 
    527                         } 
    528                         else 
    529                         { 
    530                             inactivate(item_id); 
    531                         } 
     523                        remove(item_id); 
    532524                    } 
    533525                    else 
    534526                    { 
    535                         restore(item_id);                     
     527                        inactivate(item_id); 
    536528                    } 
    537529                } 
    538530                else 
    539531                { 
    540                     activate(item_id); 
    541                 } 
    542      
     532                    if(options.allow_multiple) 
     533                    { 
     534                        restore(item_id); 
     535                    } 
     536                        else 
     537                        { 
     538                            activate(item_id); 
     539                        } 
     540                } 
     541 
    543542                return false; 
    544543            }