Skip to content

Commit

Permalink
Renamed option 'imap_ssl' into 'imap_ssl_novalidate' to better reflec…
Browse files Browse the repository at this point in the history
…t its meaning
  • Loading branch information
bwildenhain committed Mar 4, 2014
1 parent a2a5269 commit 2b1ad1e
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions settings.php
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
* License along with this library. If not, see <http://www.gnu.org/licenses/>.
*
*/
$params = array('imap_host', 'imap_port', 'imap_ssl');
$params = array('imap_host', 'imap_port', 'imap_ssl_novalidate');

OCP\Util::addscript('user_imap', 'settings');

Expand All @@ -29,7 +29,7 @@
if(isset($_POST[$param])){
OCP\Config::setAppValue('user_imap', $param, $_POST[$param]);
}
elseif('imap_ssl' == $param) {
elseif('imap_ssl_novalidate' == $param) {
// unchecked checkboxes are not included in the post paramters
OCP\Config::setAppValue('user_imap', $param, 0);
}
Expand Down
2 changes: 1 addition & 1 deletion templates/settings.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
<legend><?php echo $l->t('IMAP'); ?></legend>
<p><label for="imap_host"><?php echo $l->t('Host');?></label><input type="text" id="imap_host" name="imap_host" value="<?php echo $_['imap_host']; ?>"></p>
<p><label for="imap_port"><?php echo $l->t('Port');?></label><input type="text" id="imap_port" name="imap_port" value="<?php echo $_['imap_port']; ?>" /></p>
<p><input type="checkbox" id="imap_ssl" name="imap_ssl" value="1"<?php if ($_['imap_ssl']) echo ' checked'; ?>><label for="imap_ssl"><?php echo $l->t('Don\'t validate SSL certificate');?></label></p>
<p><input type="checkbox" id="imap_ssl_novalidate" name="imap_ssl_novalidate" value="1"<?php if ($_['imap_ssl_novalidate']) echo ' checked'; ?>><label for="imap_ssl_novalidate"><?php echo $l->t('Don\'t validate SSL certificate');?></label></p>
<input type="submit" value="<?php echo $l->t('Save'); ?>" />
</fieldset>
</form>
6 changes: 3 additions & 3 deletions user_imap.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,12 @@ class OC_USER_IMAP extends OC_User_Backend {
// cached settings
protected $imap_host;
protected $imap_port;
protected $imap_ssl;
protected $imap_ssl_novalidate;

public function __construct() {
$this->imap_host = OCP\Config::getAppValue('user_imap', 'imap_host','');
$this->imap_port = OCP\Config::getAppValue('user_imap', 'imap_port', OC_USER_BACKEND_IMAP_DEFAULT_PORT);
$this->imap_ssl = OCP\Config::getAppValue('user_imap', 'imap_ssl', false);
$this->imap_ssl_novalidate = OCP\Config::getAppValue('user_imap', 'imap_ssl_novalidate', false);
}

/**
Expand All @@ -51,7 +51,7 @@ public function checkPassword($uid, $password){
try
{
$url = $this->imap_host.":".$this->imap_port;
if ($this->imap_ssl) {
if ($this->imap_ssl_novalidate) {
$url .= "/imap/ssl/notls/novalidate-cert";
error_log("========= $url ====");
}
Expand Down

0 comments on commit 2b1ad1e

Please sign in to comment.