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

Error in Browsers console after adding tabs to Quickcreate #9344

Open
SinergiaCRM opened this issue Nov 4, 2021 · 7 comments · Fixed by #9813
Open

Error in Browsers console after adding tabs to Quickcreate #9344

SinergiaCRM opened this issue Nov 4, 2021 · 7 comments · Fixed by #9813
Assignees
Labels
Area: Module Issues & PRs related to modules that do not have specific label Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds Type: Bug Bugs within the core SuiteCRM codebase

Comments

@SinergiaCRM
Copy link
Contributor

After setting a tab configuration in a quickcreate view of a module, it appears an error in the browser Console:
Selection_339

Issue

There is some errors regarding the use of tabs in a quickcreate view.

This error may cause other JS scripts to not execute.

Expected Behavior

There shouldn't be any error.

Actual Behavior

Error appears in console

Possible Fix

Steps to Reproduce

  1. Add a panel in the QuickCreate view of the Accounts module
  2. After saving the view, change the "Panel" to "Tab" configuration
  3. Go to the Quickcreateview of Accounts (Inside an Account, the Member Organization subpanel)
  4. Open the browser console and see the error

Context

Your Environment

SuiteCRM Version used: 7.11.21
Browser name and version: Chrome
Environment name and version (e.g. MySQL, PHP 7): PHP 7
Operating System and version (e.g Ubuntu 16.04): Ubuntu

@johnM2401 johnM2401 added Area: Module Issues & PRs related to modules that do not have specific label Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds Type: Bug Bugs within the core SuiteCRM codebase labels Nov 4, 2021
@lukio
Copy link
Contributor

lukio commented Nov 3, 2022

The problem remains at 8.x suitecrm when working with legacy layout.

@lukio
Copy link
Contributor

lukio commented Nov 3, 2022

Duplicated ticket #8780

@lukio
Copy link
Contributor

lukio commented Nov 3, 2022

@SinergiaCRM I found that the name of the function selectTabOnError has been declared twice at the theme.

themes/suite8/include/DetailView/DetailView.tpl:                    let selectTabOnError = function(tab) {
themes/suite8/include/EditView/EditView.tpl:    var selectTabOnError = function(tab) {

If I change the definition of the function from let to var it does not throws the error on the javascript console.

@SinergiaCRM
Copy link
Contributor Author

Thanks @lukio. This sounds interesting. This error has been here for a while already. We will try it out.

If you are certain, you could open a PR with the proposed fix.

@lukio lukio self-assigned this Nov 10, 2022
lukio added a commit to lukio/SuiteCRM that referenced this issue Nov 10, 2022
…Quickcreate: selectTabOnError

Found that the name of the function selectTabOnError has been declared
twice at the theme SuiteP

If I change the definition of the function from let to var it does
not throws the error on the javascript console.
@lukio
Copy link
Contributor

lukio commented Nov 10, 2022

Thanks @lukio. This sounds interesting. This error has been here for a while already. We will try it out.

If you are certain, you could open a PR with the proposed fix.

@SinergiaCRM I opened a new PR with the propoosed fix.

#9813

mattlorimer pushed a commit that referenced this issue Dec 22, 2022
… selectTabOnError

Found that the name of the function selectTabOnError has been declared
twice at the theme SuiteP

If I change the definition of the function from let to var it does
not throws the error on the javascript console.
pgorod pushed a commit to pgorod/SuiteCRM that referenced this issue Feb 27, 2023
…Quickcreate: selectTabOnError

Found that the name of the function selectTabOnError has been declared
twice at the theme SuiteP

If I change the definition of the function from let to var it does
not throws the error on the javascript console.
@SinergiaCRM
Copy link
Contributor Author

Hi @lukio it seems that the problem persists in the latest version. Could it be due to something else?
image

@SinergiaCRM
Copy link
Contributor Author

SinergiaCRM commented Oct 23, 2023

@jack7anderson7 , I reopened this issue again. The error continues to occur with the change in the definition of the function selectTabOnError from let to var.
image

@SinergiaCRM SinergiaCRM reopened this Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Module Issues & PRs related to modules that do not have specific label Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds Type: Bug Bugs within the core SuiteCRM codebase
Projects
None yet
3 participants