Xefir/epubreader
Archived
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 3 Wiki Activity

Refacto all project #1

Merged
Xefir merged 18 commits from test into master 2023-06-17 23:48:37 +00:00
Conversation 0 Commits 18 Files Changed 125 +6 -4
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit f992856476 - Show all commits

5
lib/Db/BookmarkMapper.php
View File

@ -65,7 +65,6 @@ class BookmarkMapper extends ReaderMapper {
* @return ReaderEntity the newly created or updated bookmark
*/
public function set(int $fileId, string $name, string $value, ?string $type = null, ?string $content = null): ReaderEntity {
/** @var Bookmark[] $result */
$result = $this->get($fileId, $name);
if(empty($result)) {
@ -88,12 +87,14 @@ class BookmarkMapper extends ReaderMapper {
$bookmark->setContent($content ?? '');
$this->insert($bookmark);
} else {
} elseif ($result[0] instanceof Bookmark) {
$bookmark = $result[0];
$bookmark->setValue($value);
$bookmark->setContent($content ?? '');
$this->update($bookmark);
} else {
$bookmark = new Bookmark();
}
return $bookmark;

5
lib/Db/PreferenceMapper.php
View File

@ -57,7 +57,6 @@ class PreferenceMapper extends ReaderMapper {
* @return ReaderEntity the newly created or updated preference
*/
public function set(string $scope, int $fileId, string $name, string $value): ReaderEntity {
/** @var Preference[] $result */
$result = $this->get($scope, $fileId, $name);
if(empty($result)) {
@ -69,11 +68,13 @@ class PreferenceMapper extends ReaderMapper {
$preference->setValue($value);
$this->insert($preference);
} else {
} elseif ($result[0] instanceof Preference) {
$preference = $result[0];
$preference->setValue($value);
$this->update($preference);
} else {
$preference = new Preference();
}
return $preference;