Foren / Cotonti / Core Labs / Archive / Login enhancement

oc
#1 12. September 2008, 22:15
Hi,

I was wondering about what is base64 decoding for urls to redirect.

Also, is this login system enough? I mean, vbulletin style would be nice.

BTW, would it be too much if there is a user name and password input in header.tpl ?
Kilandor
#2 13. September 2008, 00:23
# oc : I was wondering about what is base64 decoding for urls to redirect.
I totally agree, this needs to happen, that's been one thing that's always bugged me.

# oc : Also, is this login system enough? I mean, vbulletin style would be nice.
About the login system, Can you be a bit more specific, on what your referring too?

# oc : BTW, would it be too much if there is a user name and password input in header.tpl ?
Yea, I just haven't got around to doing it, i've been wanting to as well
oc
#3 13. September 2008, 03:27
About the system:

No redirect if login needed. I mean if code is:
users.profile.php
if ($usr['id']<1)
	{
	header("Location: " . SED_ABSOLUTE_URL . "message.php?msg=100&".$sys['url_redirect']);
	exit;
	}

new code is something like:
if ($usr['id']<1)
	{
	require("login.php");
	exit;
	}

actually this is not very necessary, but i believe a header login (not plugin, optional feature) and cleaning up the unnecessary stuff from the redirecting function is necessary.
Kilandor
#4 13. September 2008, 17:24
Ah I see what your getting at, it might take a few changes to do something like that. I just did a similar concept with a project I just finished for someone for a admin backend. On any page, it loads login.php if not an auth user, without refreshing them, it loads that instead of the page it should.

The much more simpler solution, would be to have it base64 the URI, and redirect you to login, then when you login, it returns you to where you was.

There is a rough implementation for this in a few places in sed but not done very well or complete.

Fill out a ticket for it as an enhancment.
oc
#5 13. September 2008, 17:43
Actually after I thought a bit, I found that header login would be much more simpler and useful. And I can do that with zero session, only JS.

I'll do it instead talking and let you to decide.
oc
#6 13. September 2008, 22:11
I commited a new revision. But I see it is done here on Cotonti.com already...
Kilandor
#7 14. September 2008, 03:51
Ah yea sorry. I thought in your talking here you was talking about something different.

There's no need for a a plugin for that, it can plopped straight into the template.
oc
#8 14. September 2008, 04:16
I actually was talking what you thought but after a while, It came to me unnecessary.

To be honest, I feel like a little bit a stupid :) But more important, is this feature should be in SEDITIO? I believe it is, and actually this brings me next point: A new skin. Ok, this one is light cool and something but I can't see most things.
Kilandor
#9 14. September 2008, 06:25
Yea, we will need our own default skin ourselves. But for now, its best for most uf os to not fool with it and work with what we got.