Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incompatible with YOURLS 1.7? #5

Open
engordon opened this issue Mar 13, 2014 · 7 comments
Open

Incompatible with YOURLS 1.7? #5

engordon opened this issue Mar 13, 2014 · 7 comments

Comments

@engordon
Copy link

When the cache.php is added to the /user directory against a YOURLS 1.7 install, the first hit to the admin site goes through, but then subsequent hits are redirected to the Install page (admin/install.php)..

@jochar
Copy link

jochar commented Mar 31, 2014

Hello,
i can confirm this.... it redirects but no admin access at all

@ianbarber
Copy link
Owner

Ah right. I haven't tested against 1.7 at all. I'll make a note to take a look, but PRs welcome if you worked out what needed to change!

@jochar
Copy link

jochar commented Apr 1, 2014

Hello,
im sorry im not much of a help here as php hates me, but i can test if you want
Thanks

@hell0world
Copy link

same issue~redirect to install.php always after the first running:)

@danielzheng
Copy link

same problem, any updates here?

@ianbarber
Copy link
Owner

Sorry, I still haven't got round to looking at it - I don't use YouRLS regularly any more so its pretty low on the TODO list unfortunately!

@art-spilgames
Copy link

I found the issue, but not sure what would be the best fix for this.
Basically the YOURLS application now stores a property in the $ydb class called installed:
https://github.com/YOURLS/YOURLS/blob/master/includes/functions.php#L1019

Restoring the options via apc cache will not restore property and hence it will prompt to install the application.

The workaround for me now is to write this property on restore, but the proper way to restore is writing and restoring both the options and the installed properties in apc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants