Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NEW : use specific landing page to home menu entry #30244

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

altairis-tof
Copy link
Contributor

if specific landing page is specified, use it as dolibarr homepage menu entry link

@altairis-tof altairis-tof changed the title NEW : use specific landing page to home menu entry WIP NEW : use specific landing page to home menu entry Jul 2, 2024
@altairis-tof altairis-tof changed the title WIP NEW : use specific landing page to home menu entry NEW : use specific landing page to home menu entry Jul 2, 2024
@@ -75,9 +75,15 @@ function print_eldy_menu($db, $atarget, $type_user, &$tabMenu, &$menu, $noout =
$menu_arr = array();

// Home
$landingpage = (empty($user->conf->MAIN_LANDING_PAGE) ? (!getDolGlobalString('MAIN_LANDING_PAGE') ? '' : $conf->global->MAIN_LANDING_PAGE) : $user->conf->MAIN_LANDING_PAGE);
Copy link
Member

@eldy eldy Jul 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use of conf->global->xxx or user->conf->xxx is no more allowed.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry it was a copy/paste from main.inc.php 😉

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$user->conf-> is also not allowed.
use getDolUserString

@eldy eldy added the PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do) label Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants