Minz/quicknotes
1
0
Fork 0
mirror of https://github.com/JanGross/quicknotes.git synced 2025-11-30 23:37:16 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix unable to forget a shared note deleted by the owner. Issue #72

Browse Source
This commit is contained in:
Matias De lellis
2022-05-31 15:39:35 -03:00
parent 38860f784f
commit 073a8dbda1
5 changed files with 9 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
appinfo/routes.php
View File

@@ -19,7 +19,7 @@ return ['resources' =>
],
// Share
[
'name' => 'share#destroy',
'name' => 'share#forget',
'url' => '/share/{noteId}',
'verb' => 'DELETE'
],

2
js/script.js
View File

@@ -387,7 +387,7 @@ View.prototype = {
self._notes.forgetShare(note).done(function () {
if (self._notes.length() > 0) {
self._isotope.remove(gridnote.parent())
selg._isotope.layout();
self._isotope.layout();
self.showAll();
self.renderNavigation();
} else {

9
lib/Controller/ShareController.php
View File

@@ -54,12 +54,9 @@ class ShareController extends Controller {
*
* @param int $noteId
*/
public function destroy(int $noteId): JSONResponse {
if ($this->noteShareMapper->forgetShareByNoteIdAndSharedUser($noteId, $this->userId)) {
return new JSONResponse([], Http::STATUS_OK);
} else {
return new JSONResponse([], Http::STATUS_NOT_FOUND);
}
public function forget(int $noteId): JSONResponse {
$this->noteShareMapper->forgetShareByNoteIdAndSharedUser($noteId, $this->userId);
return new JSONResponse([]);
}
}

4
lib/Dashboard/NotesWidget.php
View File

@@ -10,8 +10,8 @@ use OCP\IURLGenerator;
class NotesWidget implements IWidget {
private IURLGenerator $url;
private IL10N $l10n;
private $url;
private $l10n;
public function __construct(IURLGenerator $url,
IL10N $l10n)

4
lib/Db/NoteShareMapper.php
View File

@@ -73,7 +73,7 @@ class NoteShareMapper extends QBMapper {
->execute();
}
public function forgetShareByNoteIdAndSharedUser(int $noteId, string $userId) {
public function forgetShareByNoteIdAndSharedUser(int $noteId, string $userId): bool {
try {
$noteShare = $this->findSharesByNoteIsAndSharedUser($noteId, $userId);
} catch (DoesNotExistException $e) {
@@ -86,7 +86,7 @@ class NoteShareMapper extends QBMapper {
/**
* @return bool
*/
public function existsByNoteAndSharedUser(int $noteId, string $userId) {
public function existsByNoteAndSharedUser(int $noteId, string $userId): bool {
try {
$this->findSharesByNoteIsAndSharedUser($noteId, $userId);
} catch (DoesNotExistException $e) {