Tickets :: Comment on [#56] IMP maintenance should apply to a configurable folder set.

Search Results: 1 of 574 [ Next> Last>> ] [ Return to Search Results ]

* Your Email Address
* Spam protection	Enter the letters below: .__ ._.. .. .. . [__) \| \ /\| \|\ / \| \_\|_ \/ \|/\\| \/
Comment	>> What is the bitwise or in these lines for? > > > > That's a good question. To be honest, I didn't write that part of > the patch and I'd ask the guy who wrote it, but he doesn't work here > anymore. Apologies on my side though, because I agree that it should > be changed. I do see some other instances of this in the > handle_folderselect() and handle_trashselect() functions in that same > file, but I think I'd be inclined to make it more > readable/trustworthy. Something like the following would be better, > IMO. > > > > // We might need to create the preference... > > $updated = FALSE; > > if (!isset($prefs->_prefs['purge_folders_list'])) { > > $updated = $updated \|\| $prefs->add('purge_folders_list', > > serialize($arr)); > > } else { > > $updated = $updated \|\| > > $prefs->setValue('purge_folders_list', serialize($arr))$ > > } > > > > What do you think ?
Attachment
Watch this ticket