Skip to content

Commit

Permalink
- Fix folder rename/delete buttons do not appear on creation of first…
Browse files Browse the repository at this point in the history
… folder (#1486468)
  • Loading branch information
alecpl committed Feb 2, 2010
1 parent f52e7a0 commit c5c3ae7
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 6 deletions.
1 change: 1 addition & 0 deletions CHANGELOG
@@ -1,6 +1,7 @@
CHANGELOG RoundCube Webmail
===========================

- Fix folder rename/delete buttons do not appear on creation of first folder (#1486468)
- Fix character set conversion fails on systems where iconv doesn't accept //IGNORE (#1486375)
- Login preformance: Create default folders on first login only
- Import contacts into the selected address book (by Phil Weir)
Expand Down
16 changes: 10 additions & 6 deletions program/js/app.js
Expand Up @@ -3368,10 +3368,13 @@ function rcube_webmail()
if (!this.gui_objects.subscriptionlist)
return false;

// find not protected folder
for (var refid in this.env.subscriptionrows)
if (this.env.subscriptionrows[refid]!=null && !this.env.subscriptionrows[refid][2])
// find not protected folder
var refid;
for (var rid in this.env.subscriptionrows)
if (this.env.subscriptionrows[rid]!=null && !this.env.subscriptionrows[rid][2]) {
refid = rid;
break;
}

var refrow, form;
var tbody = this.gui_objects.subscriptionlist.tBodies[0];
Expand All @@ -3384,10 +3387,11 @@ function rcube_webmail()
refid = replace.id;
}

if (!id || !(refrow = document.getElementById(refid)))
if (!id || !refid || !(refrow = document.getElementById(refid)))
{
// Refresh page if we don't have a table row to clone
this.goto_url('folders');
return false;
}
else
{
Expand All @@ -3413,7 +3417,7 @@ function rcube_webmail()
// set messages count to zero
if (!replace)
row.cells[1].innerHTML = '*';

if (!replace && row.cells[2] && row.cells[2].firstChild.tagName.toLowerCase()=='input')
{
row.cells[2].firstChild.value = name;
Expand Down Expand Up @@ -4175,7 +4179,7 @@ function rcube_webmail()

if (this.env.framed && window.parent)
parent.location.href = url;
else
else
location.href = url;
};

Expand Down

0 comments on commit c5c3ae7

Please sign in to comment.