Tickets :: Comment on [#10766] fb.php

* Your Email Address
* Spam protection	Enter the letters below: ..__ . ,. ..___. \|[__) \./ \|_/ \| \__\|[__) \| \| \ \|
Comment	> Just stumbled on these lines in kronolith/fb.php:45 > > // If the free/busy calendars preference is empty, default to > // the user's default_share preference, and if that's empty, > // to their username. > if (!$cal) { > $cal = 'internal_' . $prefs->getValue('default_share'); > if (!$cal) { > $cal = 'internal_' . $user; > } > } > > Am I wrong, or will the innermost if-condition never be true? > "internal_" will be stored in every case, which makes the next if > redundant!?
Attachment
Watch this ticket