Skip to content

Commit

Permalink
remove unused logs
Browse files Browse the repository at this point in the history
  • Loading branch information
ryanjpburgos committed Feb 26, 2025
1 parent ff03b43 commit ccef194
Show file tree
Hide file tree
Showing 13 changed files with 40 additions and 86 deletions.
4 changes: 1 addition & 3 deletions src/api/component-repository/categories.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
import { LIST_ECR_CATEGORIES_OK } from 'test/mocks/component-repository/categories';
import { makeRequest, METHODS } from '@entando/apimanager';
import { composeCMApiDomain } from 'helpers/apiDomainComposer';
// import { getEntandoVirtualContextFromCookies } from 'helpers/cookies';
import { getContextFromURL } from 'app-init/contextProvider';

import { getContextFromURL } from 'helpers/contextUtils';
// eslint-disable-next-line import/prefer-default-export
export const getECRCategories = () => (
makeRequest({
Expand Down
4 changes: 1 addition & 3 deletions src/api/component-repository/componentRepositories.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
import { LIST_COMPONENT_REPOSITORIES_OK } from 'test/mocks/component-repository/componentRepositories';
import { makeRequest, METHODS } from '@entando/apimanager';
import { composeCMApiDomain } from 'helpers/apiDomainComposer';
// import { getEntandoVirtualContextFromCookies } from 'helpers/cookies';
import { getContextFromURL } from 'app-init/contextProvider';

import { getContextFromURL } from 'helpers/contextUtils';
// eslint-disable-next-line import/prefer-default-export
export const getComponentRepositories = (page = { page: 1, pageSize: 10 }, params = '') => (
makeRequest(
Expand Down
3 changes: 1 addition & 2 deletions src/api/component-repository/components.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,7 @@ import {
} from 'test/mocks/component-repository/components';
import { makeRequest, METHODS } from '@entando/apimanager';
import { composeCMApiDomain } from 'helpers/apiDomainComposer';
// import { getEntandoVirtualContextFromCookies } from 'helpers/cookies';
import { getContextFromURL } from 'app-init/contextProvider';
import { getContextFromURL } from 'helpers/contextUtils';

export const getECRComponent = base64EncodedUrl => (
makeRequest({
Expand Down
3 changes: 1 addition & 2 deletions src/api/component-repository/hub.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@ import {
} from 'test/mocks/component-repository/hub';
import { makeRequest, METHODS } from '@entando/apimanager';
import { composeCMApiDomain } from 'helpers/apiDomainComposer';
// import { getEntandoVirtualContextFromCookies } from 'helpers/cookies';
import { getContextFromURL } from 'app-init/contextProvider';
import { getContextFromURL } from 'helpers/contextUtils';

export const NO_PAGE = { page: 1, pageSize: 0 };

Expand Down
4 changes: 1 addition & 3 deletions src/api/mfe.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
import { makeRequest, METHODS } from '@entando/apimanager';
import { LIST_MFE_RESPONSE_OK } from 'test/mocks/mfe';
import { composeCMApiDomain } from 'helpers/apiDomainComposer';
// import { getEntandoVirtualContextFromCookies } from 'helpers/cookies';
import { getContextFromURL } from 'app-init/contextProvider';

import { getContextFromURL } from 'helpers/contextUtils';
// eslint-disable-next-line import/prefer-default-export
export const getMfeConfigList = (params = '') => (
makeRequest({
Expand Down
5 changes: 1 addition & 4 deletions src/app-init/apiManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ import pluginsArray from 'entando-plugins';
import withAuth from 'auth/withAuth';
import getRuntimeEnv from 'helpers/getRuntimeEnv';
import { keycloak } from 'auth/keycloak/KeycloakProviderContainer';
import { getBaseUrlWithVirtualContext } from './contextProvider';
import { getBaseUrlWithVirtualContext } from 'helpers/contextUtils';

const ApiManager = ({
auth,
Expand All @@ -28,7 +28,6 @@ const ApiManager = ({
store.dispatch(clearAppTourProgress());
if (keycloak) {
const { origin } = window.location;
console.log('origin', origin);
keycloak.redirectUri = `${origin}${process.env.PUBLIC_URL || ''}${ROUTE_DASHBOARD}`;
}
auth.logout(status);
Expand All @@ -49,11 +48,9 @@ const ApiManager = ({
return;
}
const route = pathname ? pathname.replace(getBaseUrlWithVirtualContext(process.env.PUBLIC_URL), '') : null;
console.log('route', route);
const goto = auth.enabled && route && route !== ROUTE_HOME
? route
: ROUTE_DASHBOARD;
console.log('goto', goto);
history.push(goto);
}
};
Expand Down
45 changes: 0 additions & 45 deletions src/app-init/contextProvider.js

This file was deleted.

2 changes: 1 addition & 1 deletion src/app-init/router.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { createBrowserHistory } from 'history';
import { getBaseUrlWithVirtualContext } from './contextProvider';
import { getBaseUrlWithVirtualContext } from 'helpers/contextUtils';

export const history = createBrowserHistory({
basename: getBaseUrlWithVirtualContext(process.env.PUBLIC_URL),
Expand Down
20 changes: 20 additions & 0 deletions src/helpers/contextUtils.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
import getRuntimeEnv from 'helpers/getRuntimeEnv';

const getContextFromURL = (pathname) => {
const pathFragments = pathname.split('/');
const publicUrlIndex = pathFragments.findIndex(el => el === process.env.PUBLIC_URL.replace('/', ''));
const context = pathFragments[publicUrlIndex + 1];

const { ENTANDO_VIRTUAL_CONTEXTS } = getRuntimeEnv();
const virtualContexts = ENTANDO_VIRTUAL_CONTEXTS.split(',');
if (virtualContexts.includes(context)) return context;
return '';
};


const getBaseUrlWithVirtualContext = (baseUrl) => {
const virtualContext = getContextFromURL(window.location.pathname);
return virtualContext ? `${baseUrl}/${virtualContext}` : baseUrl;
};

export { getBaseUrlWithVirtualContext, getContextFromURL };
6 changes: 0 additions & 6 deletions src/helpers/getRuntimeEnv.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ const getContextFromURL = (pathname, entVirtualContexts) => {
};

const validateDomain = (domain) => {
console.log('ENTANDO_VIRTUAL_CONTEXTS', getEnvVar('ENTANDO_VIRTUAL_CONTEXTS'));
const virtualContext = getContextFromURL(window.location.pathname, getEnvVar('ENTANDO_VIRTUAL_CONTEXTS'));
if (domain) {
const isValidURL = isURL(domain, {
Expand All @@ -38,22 +37,17 @@ const validateDomain = (domain) => {
if (!isValidURL) {
throw new Error('The DOMAIN env variable is invalid.');
}
console.log('domain', domain);
let domainWithContext = domain;
if (virtualContext) {
if (domain === '/') domainWithContext = `/${virtualContext}`;
else domainWithContext = `${domain}/${virtualContext}`;
console.log('domainWithContext', domainWithContext);
}
console.log('Questo è il domain finale', domainWithContext.replace(/\/+$/, ''));
return domainWithContext.replace(/\/+$/, '');
}
if (virtualContext) {
console.log('Questo è domain finale 2', virtualContext);
return `/${virtualContext}`;
}

console.log('Questo è domain vuoto');
return '';
};

Expand Down
25 changes: 11 additions & 14 deletions src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ import '@entando/pagetreeselector/dist/css/index.css';
import '@entando/datatable/dist/css/index.css';

import 'index.scss';
import { ContextProvider } from 'app-init/contextProvider';

// init namespace for the shell
if (!window.entando) window.entando = {};
Expand All @@ -49,19 +48,17 @@ const AppRoot = (
// exporting for tests
export default ReactDOM.render(
<Provider store={store}>
<ContextProvider>
<AuthProvider store={store}>
<IntlProviderContainer>
<ApiManager store={store}>
{process.env.USE_MFE ? (
<MfeDownloadManager>
{AppRoot}
</MfeDownloadManager>
) : AppRoot}
</ApiManager>
</IntlProviderContainer>
</AuthProvider>
</ContextProvider>
<AuthProvider store={store}>
<IntlProviderContainer>
<ApiManager store={store}>
{process.env.USE_MFE ? (
<MfeDownloadManager>
{AppRoot}
</MfeDownloadManager>
) : AppRoot}
</ApiManager>
</IntlProviderContainer>
</AuthProvider>
</Provider>,
document.getElementById('root'),
);
Expand Down
3 changes: 1 addition & 2 deletions src/ui/internal-page/ContextSelect.js
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import React from 'react';
// import { setEntandoVirtualContextInCookies } from 'helpers/cookies';
import { injectIntl, intlShape } from 'react-intl';
import getRuntimeEnv from 'helpers/getRuntimeEnv';
import { getContextFromURL } from 'app-init/contextProvider';
import { getContextFromURL } from 'helpers/contextUtils';


const { ENTANDO_VIRTUAL_CONTEXTS } = getRuntimeEnv();
Expand Down
2 changes: 1 addition & 1 deletion src/ui/internal-page/VerticalMenuContainer.js
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ import MfeContainer from 'ui/app/MfeContainer';
import { getAppTourlastStep } from 'state/app-tour/selectors';
// import { selectIsPrimaryTenant } from 'state/multi-tenancy/selectors';
import { selectCurrSystemConfigAdvancedSearch } from 'state/current-system-configuration/selectors';
import { getBaseUrlWithVirtualContext } from 'app-init/contextProvider';
import { getBaseUrlWithVirtualContext } from 'helpers/contextUtils';
import getRuntimeEnv from 'helpers/getRuntimeEnv';
import ContextSelect from './ContextSelect';

Expand Down

0 comments on commit ccef194

Please sign in to comment.