ojsde / shariff

The Shariff plugin adds social media buttons to the website without compromising the privacy of website users, using Shariff (https://github.com/heiseonline/shariff) solution.
GNU General Public License v3.0
9 stars 16 forks source link

Funktioniert nicht mit OJS 3.2.1.1 #22

Closed mpbraendle closed 3 years ago

mpbraendle commented 3 years ago

Fehler:

Got error 'PHP message: PHP Fatal error: Uncaught Error: Call to a member function getSubmissionId() on null in /var/www/virtual/devojs.chimia.ch/htdocs/ojs/lib/pkp/controllers/grid/notifications/NotificationsGridCellProvider.inc.php:118\nStack trace:\n#0 /var/www/virtual/devojs.chimia.ch/htdocs/ojs/lib/pkp/controllers/grid/notifications/NotificationsGridCellProvider.inc.php(44): NotificationsGridCellProvider->_getTitle(Object(Notification))\n#1 /var/www/virtual/devojs.chimia.ch/htdocs/ojs/lib/pkp/classes/controllers/grid/GridCellProvider.inc.php(56): NotificationsGridCellProvider->getCellActions(Object(Request), Object(GridRow), Object(GridColumn))\n#2 /var/www/virtual/devojs.chimia.ch/htdocs/ojs/lib/pkp/classes/controllers/grid/GridHandler.inc.php(1104): GridCellProvider->render(Object(Request), Object(GridRow), Object(GridColumn))\n#3 /var/www/virtual/devojs.chimia.ch/htdocs/ojs/lib/pkp/classes/controllers/grid/GridHandler.inc.php(1011): GridHandler->_renderCellInternally(Object(Request), Object(GridRow), Obj...', referer: https://devojs.chimia.ch/management/settings/website

mpbraendle commented 3 years ago

Vergleich mit anderen OJS 3.2.1-1 generic plugins: alle functions sind public, bei Shariff aber nicht.

ohilbig01 commented 3 years ago

Hallo, ich hatte das Problem auch, es liegt am Rückgabewert der Funktion register() in ShariffPlugin.inc.php(). Ich versuche mal, einen pull request zu machen.

ronste commented 3 years ago
* taucht nicht auf in der Plugin-Gallery

Das Shariff-Plugin ist derzeit nicht Teil der PKP Plugin-Gallery von OJS 3.2. Es kann daher nur manuell installiert werden.

* manuelle Installation via tar.gz in /plugins/generic/ --> Plugin erscheint nicht in der Plugin-Liste

Da wir bisher kein tar.gz-Release für OJS 3.2 veröffentlicht haben ist mir momentan nicht klar woher die tar.gz-Datei kommt die Sie versucht haben zu installieren. Der letzte Release ist für OJS 3.1.1/3.1.2. Ich werde versuchen sobald wie möglich ein Release für OJS 3.2 zu veröffentlichen.

Bis dahin können Sie entweder das Git-Repo klonen und den Branch "_stable-3_21" verwenden, oder Sie laden sich die zip-Datei runter (https://github.com/ojsde/shariff -> Code -> Download zip), bennenen den Ordner "shariff-master" in "shariff" um und kopieren den extrahierten Ordner in den Ordner "plugins/generic". Das Plugin sollte dann in OJS unter "Installierte Plugins" erscheinen.

mpbraendle commented 3 years ago

Das tar.gz habe ich aus dem Master hergestellt.

"Der letzte Release ist für OJS 3.1.1/3.1.2." Ich ging davon aus, dass wenn in der README.md steht, "This plugin has been tested with OJS version 3.2.1!", dass das verwendbar sei. @ohilbig01 hat einen Pull-Request z.V. gestellt.

ronste commented 3 years ago

Das tar.gz habe ich aus dem Master hergestellt.

Ok, das sollte theoretisch (aber siehe Kommentar unten) funktionieren solange der Ordner und "shariff" umbenannt wurde.

"Der letzte Release ist für OJS 3.1.1/3.1.2." Ich ging davon aus, dass wenn in der README.md steht, "This plugin has been tested with OJS version 3.2.1!", dass das verwendbar sei. @ohilbig01 hat einen Pull-Request z.V. gestellt.

Das ist völlig korrekt. Die README.md bezieht sich auf den Stand des master-Branches im Repo und dieser läuft bereits auf verschiedenen OJS 3.2 Installationen (wurde da aber immer via git installiert). Die Releases beziehen sich immer auf die in der Releasebeschreibung angegebenen Versionen, i.e. 3.1.1/3.1.2.

Allerdings scheint es bei einem aus dem master-Branch erstellten Installationspaket Probleme bei verschiedenen Installationswegen zu geben. Wir können da leider nicht alle möglichen Kombinationen testen. Ich konnte diese Probleme jetzt bei mir nachvollziehen und werde versuchen das so schnell wie möglich zu beheben. Den Pull-Request werde ich dabei natürlich auch berücksichtigen.

mpbraendle commented 3 years ago

Danke, habe das Plugin mit den neuesten Änderungen installiert und das funktioniert nun mit OJS 3.2.1