Tickets :: Comment on [#9095] Fix dimp_show

* Your Email Address
* Spam protection	Enter the letters below: .__ .__ . .. .. . \| \[__)\| \|\|\/\|\ / \|__/\| \\|/\\|\| \| \/
Comment	> I've found the solution to this problem... When IMP uses database > backend for preferences, IMP saves just some preferences (I think > "editable preferences", locked = false) and uses prefs file to get > others. However when IMP uses ldap backend for preferences, IMP saves > all of them in LDAP and it creates a trouble with dimp_show_preview > pref. > > In "imp/templates/dimp/javascript_defs_dimp.php" line 70 > 'preview_pref' => $GLOBALS['prefs']->getValue('dimp_show_preview') > === true ? 'horiz' : $GLOBALS['prefs']->getValue('dimp_show_preview'), > This never will be true because LDAP returns a string type and not a > boolean type. > > I've made a patch, can you guys analyze and give a feedback if it is > a better way to solve it?
Attachment
Watch this ticket