otetard / ownpad

Ownpad is a Nextcloud application that allows to create and open Etherpad and Ethercalc documents.
GNU Affero General Public License v3.0
57 stars 24 forks source link

Unable to create public pad if Etherpad API is not used #75

Closed jeyca closed 5 years ago

jeyca commented 5 years ago

Iam using NC 14.04 and ownpad 0.6.8. I managed to set up ownpad with mime settings and so on, but when I open my new created pad, it only shows me the NC header, a black bar that seems to come from ownpad (with file name and link to pad when I hover it) but below there only a white screen and nothing else. I even cant write in this screen.

screenshot_2018-12-07_13-20-52

And when I lookup the Network connections in Firefox developer, I see that there is no connection established (or even tried to establish) to the pad server, so its actually not possible that there is anything shown :)

What is possibly going wrong here? Why is the pad not loaded?

LLynx commented 5 years ago

nearly same here: try to generate a new pad with

bildschirmfoto 2018-12-14 um 00 21 48

but can't see this file / try in file's overview after enter the name and confirm

config NC 14.0.4 with PHP 7.2 and ownpad 0.6.9

nextcloud.log said {"reqId":"###","level":3,"time":"2018-12-13T23:10:29+00:00","remoteAddr":"###","user":"###","app":"PHP","method":"POST","url":"\/index.php\/apps\/ownpad\/ajax\/v1.0\/newpad","message":"Undefined variable: eplInstance at ###PATH###/apps\/ownpad\/lib\/Service\/OwnpadService.php#59","userAgent":"Mozilla\/5.0 (Macintosh; Intel Mac OS X 10.13; rv:64.0) Gecko\/20100101 Firefox\/64.0","version":"14.0.4.2"}

here: Undefined variable: eplInstance at ###PATH###/apps\/ownpad\/lib\/Service\/OwnpadService.php#59

otetard commented 5 years ago

This bug seems to have been introduced by 22ea59c92a5dddaf6717d856e6860dd258154e8b. Fix should be easy (we just need to add a check to be able to create pad if API is not allowed).

jeyca commented 5 years ago

@otetard can you explain what the actual problem is and how to fix it for my setup? Is it, because my padserver does not allow api access? I could actually change that as well

otetard commented 5 years ago

That is a regression introduced in v0.6.9. You could either fix the issue (by analyzing 22ea59c92a5dddaf6717d856e6860dd258154e8b which introduced the bug) or rollback your version to 0.6.8.

I’ll try to find some time to fix the issue in a new version in the following days.

otetard commented 5 years ago

That issue should be fixed by 14666c00d758d2ed93f0dca514b87221ebe983eb. A new release will be pushed shortly to the Nextcloud appstore.