aboutsummaryrefslogtreecommitdiff
path: root/app/Models
diff options
context:
space:
mode:
authorGravatar Marien Fressinaud <dev@marienfressinaud.fr> 2015-01-31 14:48:35 +0100
committerGravatar Marien Fressinaud <dev@marienfressinaud.fr> 2015-01-31 14:48:35 +0100
commitc29b6d4c226cf9c45d7d372ee4bce3bcbcca99a6 (patch)
tree9a8f2be9185cd110988bf8a687aaa316c9145db9 /app/Models
parente92acfd8227bc3ac94e9039343ff76c20facf29a (diff)
parente91b72b63cd11ae3c4f59e48439e93955242c673 (diff)
Merge branch 'dev' into beta
Conflicts: README.fr.md README.md
Diffstat (limited to 'app/Models')
-rw-r--r--app/Models/ConfigurationSetter.php12
-rw-r--r--app/Models/Feed.php11
2 files changed, 9 insertions, 14 deletions
diff --git a/app/Models/ConfigurationSetter.php b/app/Models/ConfigurationSetter.php
index f1c465c7c..eeb1f2f4c 100644
--- a/app/Models/ConfigurationSetter.php
+++ b/app/Models/ConfigurationSetter.php
@@ -153,11 +153,11 @@ class FreshRSS_ConfigurationSetter {
}
private function _shortcuts(&$data, $values) {
- foreach ($values as $key => $value) {
- if (isset($data['shortcuts'][$key])) {
- $data['shortcuts'][$key] = $value;
- }
+ if (!is_array($values)) {
+ return;
}
+
+ $data['shortcuts'] = $values;
}
private function _sort_order(&$data, $value) {
@@ -210,9 +210,7 @@ class FreshRSS_ConfigurationSetter {
private function _mark_when(&$data, $values) {
foreach ($values as $key => $value) {
- if (isset($data['mark_when'][$key])) {
- $data['mark_when'][$key] = $this->handleBool($value);
- }
+ $data['mark_when'][$key] = $this->handleBool($value);
}
}
diff --git a/app/Models/Feed.php b/app/Models/Feed.php
index 86cbb783e..5ce03be5d 100644
--- a/app/Models/Feed.php
+++ b/app/Models/Feed.php
@@ -240,19 +240,16 @@ class FreshRSS_Feed extends Minz_Model {
$subscribe_url = $feed->subscribe_url(true);
}
+ $clean_url = url_remove_credentials($subscribe_url);
if ($subscribe_url !== null && $subscribe_url !== $url) {
- if ($this->httpAuth != '') {
- // on enlève les id si authentification HTTP
- $subscribe_url = preg_replace('#((.+)://)((.+)@)(.+)#', '${1}${5}', $subscribe_url);
- }
- $this->_url($subscribe_url);
+ $this->_url($clean_url);
}
if (($mtime === true) ||($mtime > $this->lastUpdate)) {
- syslog(LOG_DEBUG, 'FreshRSS no cache ' . $mtime . ' > ' . $this->lastUpdate . ' for ' . $subscribe_url);
+ Minz_Log::notice('FreshRSS no cache ' . $mtime . ' > ' . $this->lastUpdate . ' for ' . $clean_url);
$this->loadEntries($feed); // et on charge les articles du flux
} else {
- syslog(LOG_DEBUG, 'FreshRSS use cache for ' . $subscribe_url);
+ Minz_Log::notice('FreshRSS use cache for ' . $clean_url);
$this->entries = array();
}