|
|
|
J!1.5, CB and Joomfish 2.0 - no login
|
|
Date: 2008/04/03 16:49
|
By: rouven
|
Status: User
|
|
|
Karma: 0  
|
|
Fresh Joomlapolitan  | Posts: 5 |   | |
|
I don't want to make crosspostings, but I don't know exactly where the problem is. I am currently trying to get J!1.5 with CB 1.1 (mod_cblogin html escape fix already done) and Joomfish 2.0 Beta running. It seems the mod_cblogin is not working with the Joomfish Abstraction Layer. You can find this issue report also at http://www.joomfish.net/forum/viewtopic.php?f=28&p=3393
cheers,
Rouven
|
|
The administrator has disabled public write access. |
|
|
|
|
Re:J!1.5, CB and Joomfish 2.0 - no login
|
|
Date: 2008/04/04 09:34
|
By: rouven
|
Status: User
|
|
|
Karma: 0  
|
|
Fresh Joomlapolitan  | Posts: 5 |   | |
|
from my point of view the login string in action field is looking good: <form action="http://my.org/index.php/lang-en/component/comprofiler/?task=login" method="post" id="mod_loginform" style="margin: 0px;">
I don't know what can cause the problems. Anyone with ideas?
|
|
The administrator has disabled public write access. |
|
|
|
|
Re:J!1.5, CB and Joomfish 2.0 - no login
|
|
Date: 2008/04/04 09:56
|
By: rouven
|
Status: User
|
|
|
Karma: 0  
|
|
Fresh Joomlapolitan  | Posts: 5 |   | |
|
I have done more debugging and discovers that line 1411 in comprofiler.php seems to be the turnpoint: if ( $_CB_database->loadObject( $row ) && cbHashPassword( $passwd2, $row ) ) {
The Database Query returns false, and therefor the login function exits. The cbHashPassword function is true.
Now I need help, because I am not that familar with the DB Api in CB.
Cheers,
Rouven
|
|
The administrator has disabled public write access. |
|
|
|
Re:J!1.5, CB and Joomfish 2.0 - no login
|
|
Date: 2008/04/04 11:30
|
By: beat
|
Status: Admin
|
|
|
Karma: 238  
|
|
Admin  | Posts: 3997 |   | |
|
I took a quick look at this and think there is a bug in JoomFish 2.0 beta database of the joomfish plugin:
in file: jfdatabase/jfdatabase_decorator.php line 313, you have:
| Code: | /**
* Override query in order to extract ref tables data
*
* @return n/a
*/
function query() {
$this->_db->query();
$this->setRefTables();
return $this->_cursor;
}
|
and imho it should be:
| Code: | /**
* Override query in order to extract ref tables data
*
* @return n/a
*/
function query() {
$this->_db->query();
$this->setRefTables();
return $this->_db->_cursor;
}
|
CB's database engine needs that returned value of query() to be correctly the real mysql cursor, and with the missing "->_db" in the last line of code, it doesn't happen.
Could you please test the proposed fix above, and report back here the result, as well as the fix proposal to joomFish team, if it works ?
Edit: the fix above is a quick fix, not tested, no warranties. But the value _cursor returned by query (and maybe other places ?) in joomFish seems to be a true/false in joomFish, while it should be either false or a real mysql/mysqli cursor everywhere.
Edit: other CB functions will be affected by that bug as well, not only login. So it's really a needed fix. I'm stickying this topic.
Thanks,
Post edited by: beat, at: 2008/04/04 11:36
Post edited by: beat, at: 2008/04/04 11:37 Beat - Developer on Community Builder core Team - If you like CB and this forum, you will love Nick's CB 1.1 reference manual ! : Click here to Get it now  - Would like to help us move faster ? Get it, and/or help us spend more time coding by helping others in this forum, many thanks
|
|
The administrator has disabled public write access. |
|
|
|
|
Re:J!1.5, CB and Joomfish 2.0 - no login
|
|
Date: 2008/04/04 12:38
|
By: rouven
|
Status: User
|
|
|
Karma: 0  
|
|
Fresh Joomlapolitan  | Posts: 5 |   | |
|
|
I am very sorry, but this fix is not working. I will revert all my recent changes and report...
|
|
The administrator has disabled public write access. |
|
|
|
|
Re:J!1.5, CB and Joomfish 2.0 - no login
|
|
Date: 2008/04/05 08:40
|
By: rouven
|
Status: User
|
|
|
Karma: 0  
|
|
Fresh Joomlapolitan  | Posts: 5 |   | |
|
|
with fresh install, too. if this is a possible bugfix it is not the only one needed...
|
|
The administrator has disabled public write access. |
|
|