Closed GoogleCodeExporter closed 8 years ago
Thanks for helping out.
1. The admin panel on the repo at the moment is just the one from v2, left
there for reference. We'll be building a new admin panel, but the priorities
are more on making the actual shortener work, as we need to do that so we know
what options etc to include in the admin panel.
2. That's odd. What page are you visiting when that comes up? Just index.php?
3. Yes, that needs to be fixed. I'll do so shortly.
4. The API link should link to
http://site.si/api/create.php?url=http://example.org/
Example.org is just there so visitors can understand the API usage with a
sample URL. I'll probably make a page explaining the API eventually, and link
it to there then. If the link is actually linking to example.org/api... then
you need to edit includes/config.php and change the SITE_URL config option.
Hope this helps.
Original comment by he...@phurlproject.org
on 29 May 2012 at 1:53
1. ok, will wait for it. ;)
2. Yes, when opening index.php it gives those 2 errors 2 times - each 2 times.
3. ok
4. the point is that it`s already set - define('SITE_TITLE', 'site.si');
There are <li><a href="/api/create.php?url=http://example.org/">API</a></li>
in includes/themes/default/header.php
Original comment by edzu...@gmail.com
on 29 May 2012 at 2:21
2. That's odd, I'll look into that.
4. It should link to the right place. It's just linking to the API so it
shortens http://example.org/. Once on the API page, you can change example.org
to whatever in the address bar and it will shorten the URL you put. Example.org
is just there to demonstrate how to use the API.
Original comment by he...@phurlproject.org
on 29 May 2012 at 2:26
4. ohh ok, thanks for info.
Original comment by edzu...@gmail.com
on 29 May 2012 at 2:56
2 & 3 are now both fixed in latest commit.
Admin panel will be a while yet I'm afraid :)
Original comment by he...@phurlproject.org
on 29 May 2012 at 6:46
Thanks! Will check tomorrow. By the way - how to do a correct hg update if
phurl is located in folder called dev ?
And you can let me know if you need to Test if some things working well. ;)
Original comment by edzu...@gmail.com
on 29 May 2012 at 7:42
Probably best to clone it, and keep a clone as an up to date 'blank' copy, then
copy those files across to wherever you want to test phurl 3. This is what I
do.. local copy -> test on server -> commit to repo.
Thanks for the offer, I'll let you know if anything crops up :)
Original comment by he...@phurlproject.org
on 29 May 2012 at 9:09
Just updated to new clone. Got php notice when shorting url (writing some
random url and click shorten) - [Wed May 30 09:48:11 2012] [error] [client
85.254.*] PHP Notice: Undefined index: a in */api/create.php on line 8,
referer: http://site.si/index.php
Original comment by edzu...@gmail.com
on 30 May 2012 at 6:52
[deleted comment]
The first error there is something I need to fix - will sort that presently.
The second error is more something I'll have to look into, for the sake of
avoiding the error. It's basically because now, instead of using mod_rewrite,
Phurl uses ErrrorDocument 404 in .htaccess to forward URLs. Whilst this works
great and doesn't require any additional mods, it does log that error (because
we're essentially tricking it by forwarding 404s to the redirect page which
then redirects to the target URL). I'll see if there's a way to stop it logging
the error.
Original comment by he...@phurlproject.org
on 30 May 2012 at 9:02
ok, let me know there if fix comes out ;)
Original comment by edzu...@gmail.com
on 30 May 2012 at 9:07
Fixed in latest commit :)
Original comment by he...@phurlproject.org
on 31 May 2012 at 8:07
still have this issue [Thu May 31 12:37:48 2012] [error] [client 85.254.*] File
does not exist: */dev/53VAc-
Original comment by edzu...@gmail.com
on 31 May 2012 at 9:38
Yes sorry... I fixed:
[error] [client 85.254.*] PHP Notice: Undefined index: a in */api/create.php
on line 8, referer: http://site.si/index.php
The other error is just due to the way Phurl 3 does redirects. It's just
because /53VAc- does indeed not exist, as it's being redirected to go.php. I
did some quick research into this but it turns out there's not really a way to
prevent it. If it really bugs you, I'm sure there's a way you can configure
your PHP installation to ignore it, but script side, there is nothing I can do.
Original comment by he...@phurlproject.org
on 31 May 2012 at 9:42
i`m wondering why /53VAc (without - ) not giving the same error?
Original comment by edzu...@gmail.com
on 31 May 2012 at 9:48
Try shortening another URL and visiting that.. you should see the error.
I think it's because your browser has cached the redirect so it's not actually
hitting the page.
Original comment by he...@phurlproject.org
on 31 May 2012 at 9:56
Original issue reported on code.google.com by
edzu...@gmail.com
on 29 May 2012 at 1:41