use \OCP\AppFramework\Controller;
use \OCP\AppFramework\Http\JSONResponse;
+use \OCA\Grauphel\Lib\Client;
use \OCA\Grauphel\Lib\NoteStorage;
use \OCA\Grauphel\Lib\OAuth;
+use \OCA\Grauphel\Lib\OAuthException;
use \OCA\Grauphel\Lib\Dependencies;
use \OCA\Grauphel\Lib\Response\ErrorResponse;
*/
class ApiController extends Controller
{
- /**
- * constructor of the controller
+ /**
+ * constructor of the controller
*
- * @param string $appName Name of the app
- * @param IRequest $request Instance of the request
- */
- public function __construct($appName, \OCP\IRequest $request, $user)
+ * @param string $appName Name of the app
+ * @param IRequest $request Instance of the request
+ */
+ public function __construct($appName, \OCP\IRequest $request, $user)
{
parent::__construct($appName, $request);
$this->user = $user;
$urlGen = $deps->urlGen;
try {
- $provider = new \OAuthProvider();
+ $provider = OAuth::getProvider();
$oauth->registerHandler($provider)
->registerAccessTokenHandler($provider);
$provider->checkOAuthRequest(
$token = $deps->tokens->load('access', $provider->token);
$username = $token->user;
- } catch (\OAuth_Exception $e) {
+ } catch (OAuthException $e) {
return new ErrorResponse($e->getMessage());
} catch (\OAuthException $e) {
if ($e->getCode() != OAUTH_PARAMETER_ABSENT) {
'api-version' => '1.0',
);
+ $cl = new Client();
+ $client = $cl->getClient();
+ if ($client !== false) {
+ $data['oauth_authorize_url'] .= '?client=' . urlencode($client);
+ }
+
if ($authenticated) {
$data['user-ref'] = array(
'api-ref' => $urlGen->getAbsoluteURL(
'grauphel.api.user', array('username' => $username)
)
),
- 'href' => null,//FIXME
+ 'href' => null,
);
}
)
)
);
- $syncdata = $this->notes->loadSyncData($username);
+ $syncdata = $this->notes->loadSyncData();
$data = array(
'user-name' => $username,
'grauphel.api.notes', array('username' => $username)
)
),
- 'href' => null,
+ 'href' => $this->deps->urlGen->getAbsoluteURL(
+ $this->deps->urlGen->linkToRoute('grauphel.gui.index')
+ ),
),
'latest-sync-revision' => $syncdata->latestSyncRevision,
'current-sync-guid' => $syncdata->currentSyncGuid,
)
)
);
- $syncdata = $this->notes->loadSyncData($username);
- return $this->fetchNotes($username, $syncdata);
+ $syncdata = $this->notes->loadSyncData();
+ return $this->fetchNotes($syncdata);
}
/**
)
)
);
- $syncdata = $this->notes->loadSyncData($username);
-
+ $syncdata = $this->notes->loadSyncData();
+
$res = $this->handleNoteSave($username, $syncdata);
if ($res instanceof \OCP\AppFramework\Http\Response) {
return $res;
}
- return $this->fetchNotes($username, $syncdata);
+ return $this->fetchNotes($syncdata);
}
- protected function fetchNotes($username, $syncdata)
+ protected function fetchNotes($syncdata)
{
$since = null;
if (isset($_GET['since'])) {
}
if (isset($_GET['include_notes']) && $_GET['include_notes']) {
- $notes = $this->notes->loadNotesFull($username, $since);
+ $notes = $this->notes->loadNotesFull($since);
} else {
- $notes = $this->notes->loadNotesOverview($username, $since);
+ $notes = $this->notes->loadNotesOverview($since);
}
//work around bug https://bugzilla.gnome.org/show_bug.cgi?id=734313
return;
}
- //note that we have more data in $arPut than just our JSON
- // request object merges it with other data
+ //Note that we have more data in $arPut than just our JSON.
+ // The request object merges it with other data.
$arPut = $this->request->put;
//structural validation
}
//update
- ++$syncdata->latestSyncRevision;
- foreach ($arPut['note-changes'] as $noteUpdate) {
- //owncloud converts object to array, so we reverse
- $noteUpdate = (object) $noteUpdate;
-
- $note = $this->notes->load($username, $noteUpdate->guid);
- if (isset($noteUpdate->command) && $noteUpdate->command == 'delete') {
- $this->notes->delete($username, $noteUpdate->guid);
- } else {
- $this->notes->update(
- $note, $noteUpdate, $syncdata->latestSyncRevision
- );
- $this->notes->save($username, $note);
+ \OC_DB::beginTransaction();
+ try {
+ ++$syncdata->latestSyncRevision;
+ foreach ($arPut['note-changes'] as $noteUpdate) {
+ //owncloud converts object to array, so we reverse
+ $noteUpdate = (object) $noteUpdate;
+
+ $note = $this->notes->load($noteUpdate->guid);
+ if (isset($noteUpdate->command) && $noteUpdate->command == 'delete') {
+ $this->notes->delete($noteUpdate->guid);
+ } else {
+ $this->notes->update(
+ $note, $noteUpdate, $syncdata->latestSyncRevision
+ );
+ $this->notes->save($note);
+ }
}
- }
- $this->notes->saveSyncData($username, $syncdata);
+ $this->notes->saveSyncData($syncdata);
+ \OC_DB::commit();
+ } catch (\DatabaseException $e) {
+ \OC_DB::getConnection()->rollBack();
+ throw $e;
+ }
}
/**
* @NoCSRFRequired
* @PublicPage
*/
- public function note()
+ public function note($username, $guid)
{
- //FIXME
- $deps = Dependencies::get();
- $username = $deps->urlGen->loadUsername();
- $guid = $deps->urlGen->loadGuid();
- $oauth = new \OAuth();
- $oauth->setDeps($deps);
- $oauth->verifyOAuthUser($username, $deps->urlGen->note($username, $guid));
+ $this->verifyUser(
+ $username,
+ $this->deps->urlGen->getAbsoluteURL(
+ $this->deps->urlGen->linkToRoute(
+ 'grauphel.api.note',
+ array('username' => $username, 'guid' => $guid)
+ )
+ )
+ );
- $note = $this->notes->load($username, $guid, false);
+ $note = $this->notes->load($guid, false);
if ($note === null) {
header('HTTP/1.0 404 Not Found');
header('Content-type: text/plain');
exit(1);
}
- $data = array('note' => array($note));
- $deps->renderer->sendJson($data);
+ return new JSONResponse($note);
}
/**
*/
protected function verifyUser($username, $curUrl)
{
- if ($this->user !== null && $this->user->getUID() == $username) {
+ if ($this->user !== null && $this->user->getUid() == $username) {
+ $this->notes->setUsername($username);
return true;
}
$oauth = new OAuth();
$oauth->setDeps($this->deps);
$oauth->verifyOAuthUser($username, $curUrl);
+
+ $this->notes->setUsername($username);
+ return true;
}
}
?>