by eggsurplus

Control what your users can access and save time, money, and frustrations. Lock down sensitive data in SuiteCRM to specific groups or teams. Supports unlimited assigned users, unlimited group assignments to records, custom layouts for each group, login/sudo capabilities and much more.

Free 30 day trial
Try it Now

#4734 - 500 crash. Non-static method Relationship::get_other_module() cannot be called statically

Open Bug? created by t Verified Purchase a year ago

modules/SecurityGroups/SecurityGroup.php contains a mistake at 351 where the Relationship get_other_module method is being called statically. This method is non-static and is causing the CRM to crash.

a fix is to either make the method a static method and update it everywhere it is used. or to update SecurityGroup.php to create an instance of a relationship and call get_other_module non-statically

  1. t member avatar

    t Verified Purchase

    a year ago

    replacing $rel_module = Relationship::get_other_module($_REQUEST['relate_to'], $focus_module_dir, $focus->db); with $rel_module = (new Relationship())->get_other_module($_REQUEST['relate_to'], $focus_module_dir, $focus->db); solves the problem

    • eggsurplus member avatar

      eggsurplus Provider Affiliate

      a year ago

      Yes, thank you for reporting this. I recommend disabling these notices in your php.ini for now. For example:

      error_reporting = E_ALL & ~E_WARNING & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT
      
    • t member avatar

      t Verified Purchase

      a year ago

      Hi Thankyou eggsurplus for your reply changing error_reporting configuration wouldn't prevent a PHP application from crashing due to a fatal error. Your suggestion would only hide PHP error messages.

    • eggsurplus member avatar

      eggsurplus Provider Affiliate

      a year ago

      That's not true.

    • eggsurplus member avatar

      eggsurplus Provider Affiliate

      a year ago

      To clarify, typically installs have display_errors set on which will cause apps to crash if there are any notices, warnings, etc based on the error_reporting config.

This case is public. Please leave out any sensitive information such as URLs, passwords, etc.
Saving Comment Saving Comment...
Rating
  • "The add-on itself was already a must for my SuiteCRM, which was missing this very important security feature. However, what surprised me the most was ..." - Davint

    Read More Reviews