Mailbox message selection error 7.1.1

Hi,

After upgrading to 7.1.1, I realized that the checkboxes are not working in the mailbox. When you click a checkbox next to a message, the input field should be updated as checked in html but it does not. However, if I click select all then I see that each input field is updated as checked in html. I am guessing something is missing in mailbox.js. Probably an event listener for checkboxes.

If this is not a bug and only happens to me, can someone please tell me how to fix it? 

Best.

I have to got to each email and delete it.. can't select all and delete.. So not just you..

Well I think I found a solution. So if anyone interested, here is change:

In mailbox.js you have functions that removes, hides and restore messages. Each of these functions retrieve selected checkboxes via the following jquery:

var oCheckBoxes = $("." + sContainer + " input:checkbox:enabled[checked]").each(function(){

    if ($(this).attr('owner'))

    {

    iValue = $(this).attr('owner').replace(/[a-z]{1,}/i, '');

    if ( iValue )

    sMembersId += iValue + ',';

    }

    });

Unfortunately, this jquery is wrong and it has to be changed for all functions that contain it to the following:

var oCheckBoxes = $("." + sContainer + " input:checkbox:enabled").each(function(){

if ($(this).attr('checked')) {

iValue = $(this).attr('value').replace(/[a-z]{1,}/i, '');

if ( iValue )

sMessagesId += iValue + ',';

}

});

I guess that Dolphin7.1.1 is not perfect to install on a production site.  I will have to pause my site for now. 

Can't reproduce the problem, messages are selected and deleted without any problems.

Maybe you are experiencing this problem under some particular circumstances ? can you reproduce on on demo.boonex.com ?

It looks like it goes pretty deeply. If you uncheck the letter or greeting checkboxes, you can't re-check them. Here are the changes I've found so far that need to be made. I will list the file, and the function the code is found in, and approximate line numbers:

/inc/mail_box.js:

in the hideDeletedMessages function, around line 46, replace:

var oCheckBoxes = $("." + sContainer + " input:checkbox:enabled[checked]").each(function(){

with

var oCheckBoxes = $("." + sContainer + " input:checkbox:checked").each(function(){

in the deleteMessages function, around line 79, replace:

var oCheckBoxes = $("." + sContainer + " input:checkbox:enabled[checked]").each(function(){

with

var oCheckBoxes = $("." + sContainer + " input:checkbox:checked").each(function(){

in the hideDeletedMessages function, around line 133, replace:

var oCheckBoxes = $("." + sContainer + " input:checkbox:enabled[checked]").each(function(){

with

var oCheckBoxes = $("." + sContainer + " input:checkbox:checked").each(function(){

as mentioned before, in the hideDeletedMessages function, around line 167, replace:

var oCheckBoxes = $("." + sContainer + " input:checkbox:enabled[checked]").each(function(){
  if ($(this).attr('owner'))
   {
   iValue = $(this).attr('owner').replace(/[a-z]{1,}/i, '');

with

var oCheckBoxes = $("." + sContainer + " input:checkbox:checked").each(function(){
 if ($(this).attr('checked'))
  {
  iValue = $(this).attr('value').replace(/[a-z]{1,}/i, '');

/templates/base/mail_init_box.html (and/or any custom skin file with the same name):

in the MailBoxMessages.prototype.markMessages funtion, around line 73, replace:

var oCheckBoxes = $(".messages_container input:checkbox:enabled[checked]").each(function(){

with

var oCheckBoxes = $(".messages_container input:checkbox:checked").each(function(){

in the MailBoxMessages.prototype.getTypifiedPage function, around line 116, replace:

var oCheckBoxes = $(".top_settings_block .left_section input:checkbox:enabled[checked]").each(function()

with

var oCheckBoxes = $(".top_settings_block .left_section input:checkbox:checked").each(function()

Be sure to clear the Dolphin cache afterwards, so that the changes will take effect. Hopefully boonex will add this patch soon into 7.1.2

 Alex, I can reproduce it on demo.boonex.com. I reproduced it using Google Chrome (25.0.1364.172 m), Firefox (19.0.2), Opera (12.14 build 1738), Safari (5.1.7), and IE 9 (9.0.8112.16421 64-bit) on Windows 7. Try sending a message or greeting (or anything) then go into the outbox, and from the list click an individual checkbox (not the all or none checkboxes at the bottom, after you click one of those the bug won't appear) then delete and you will see the bug. Also, uncheck the greeting or letter checkbox and the top and then recheck and you will see one of the other bugs.

The fix I posted above fixes the problems in all 5 browsers I mentioned.

Thank you for the detailed description and provided solution:

http://www.boonex.com/trac/dolphin/ticket/3110

Alex, no problem. I've found a few more issues in the core code that I'll be posting in a separate thread in a few minutes.

change set 17539 did not work for me....

I must still click the select all link first then deselect the emails i want to keep.

I can also deselect all the emails after clicking select all then i am able to select any number of emails i want to delete with the check boxes.

I can not just go select any box i want to then click delete.  i must click select all first .... 

Any other idea that i could try to fix this ?

hi i am using dolphin 7.1.1

did as the instruction Draxxon (thank to him)

and its fixed! 

 Hello

We've rechecked the fix and there is no need to click "Select All" first to delete a single mail after it was applied. Did you clear cache in Dolphin and in your browser before testing?

Moreover the solution provided by Draxxon is a main part of the fix and it's working for the others posted here.

Let me run down the forum post again and make sure all the code and change set's are in place.  

Yes, i have cleared all cache many times.  

I'll get back to you in a few to let you know. 

Cancel my post above.  Fired up a new virtual machine and the delete function works as expected.  Sorry for the confusion.