J!1.5, CB and Joomfish 2.0 - no login

11 years 7 months ago #60360 by rouven
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 www.joomfish.net/forum/viewtopic.php?f=28&p=3393

cheers,

Rouven
11 years 7 months ago #60391 by rouven
from my point of view the login string in action field is looking good: <form action=" 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?
11 years 7 months ago #60394 by rouven
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
11 years 7 months ago #60398 by beat
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:1] /**
* Override query in order to extract ref tables data
*
* @return n/a
*/
function query() {
$this->_db->query();
$this->setRefTables();
return $this->_cursor;
}
[/code:1]

and imho it should be:

[code:1] /**
* Override query in order to extract ref tables data
*
* @return n/a
*/
function query() {
$this->_db->query();
$this->setRefTables();
return $this->_db->_cursor;
}
[/code:1]

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 - Community Builder Team Member

Before posting on forums: Read FAQ thoroughly -- Help us spend more time coding by helping others in this forum, many thanks :)
CB links: Our membership - CBSubs - Templates - Hosting - Forge - Send me a Private Message (PM) only for private/confidential info
11 years 7 months ago #60401 by rouven
I am very sorry, but this fix is not working. I will revert all my recent changes and report...
11 years 7 months ago #60436 by rouven
with fresh install, too. if this is a possible bugfix it is not the only one needed...
Moderators: beatnantkrileon
Time to create page: 0.417 seconds
Facebook Twitter Google LinkedIn