HackerOne users: Testing against this community violates our program's Terms of Service and will result in your bounty being denied.

Vanilla 2021.009 no categories found

12346»

Comments

  • R_JR_J Ex-Fanboy Munich Admin
    pinchiesMrCaspanpioc34
  • MrCaspanMrCaspan
    edited August 11

    You can also double test by changing line 888 in the file /applications/dashboard/models/class.permissionmodel.php and add var_dump($permissions);exit; to it

    Example

    Original

    887        $permissions = $sql->get()->resultArray();
    888        return $permissions;
    

    To This

    887        $permissions = $sql->get()->resultArray();
    888        var_dump($permissions);exit;
    889        return $permissions;
    

    It will then do a dump of the $permissions array to ensure that Vanilla is reading the values properly using what ever PDO settings vanilla is using.


    And again please note this fixes one issue... there might be more but it fixes the Categories not showing up there might be other bugs related to this that are not solved by this.. Just a peace of the puzzle ;)

    pinchies
  • I suspect this has been an issue for a long time... and I hope that the essential info about using mysqlnd can be added to the installation documentation?

    I actually previously submitted a PR to Vanilla forum for a very similar database type issue, quite a while back -- except from memory that one was about a string/bool mismatch.

  • Also to note it might be a good idea to keep the nd_mysqli setting that we tested before as this affects some PDO statements also

    pinchies
  • Ok, so the final recommendation is:


    MrCaspan
  • MrCaspanMrCaspan
    edited August 11

    I suspect it has been also but the older code the coders were using == not === in their statements, It actually explains a lot of small weird bugs I was having with forms showing as not read but they were read. It could have caused all kinds of just small issues. I agree this should be added to the documentation as a requirement for Vanilla as well there shoudl be coded added to the installer that tests to ensure you are not a victim of this issue before installing.

    pinchies
  • pinchiespinchies New
    edited August 11

    Yes, it would be even better if the installer code tested for it and gave a warning!

  • It's a MySQL statement you would run it on your MySQL server

Sign In or Register to comment.