QT/superset2
1
0
Fork 0
mirror of https://github.com/apache/superset.git synced 2026-05-09 18:05:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: Unable to sync columns when database or dataset name contains + (#25390)

Browse Source 
(cherry picked from commit dbe0838f8f)
This commit is contained in:
mapledan
2023-10-04 02:15:01 +08:00
committed by Michael S. Molina
parent 4ad2a05333
commit ae700d13cf
2 changed files with 3 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
superset-frontend/src/components/Datasource/DatasourceEditor.jsx
View File

@@ -762,9 +762,7 @@ class DatasourceEditor extends React.PureComponent {
database_name:
datasource.database.database_name || datasource.database.name,
schema_name: datasource.schema,
table_name: datasource.table_name
? encodeURIComponent(datasource.table_name)
: datasource.table_name,
table_name: datasource.table_name,
normalize_columns: datasource.normalize_columns,
};
Object.entries(params).forEach(([key, value]) => {
@@ -773,7 +771,7 @@ class DatasourceEditor extends React.PureComponent {
params[key] = null;
}
});
const endpoint = `/datasource/external_metadata_by_name/?q=${rison.encode(
const endpoint = `/datasource/external_metadata_by_name/?q=${rison.encode_uri(
params,
)}`;
this.setState({ metadataLoading: true });

2
superset-frontend/src/components/Datasource/DatasourceEditor.test.jsx
View File

@@ -75,7 +75,7 @@ describe('DatasourceEditor', () => {
setTimeout(() => {
expect(fetchMock.calls(DATASOURCE_ENDPOINT)).toHaveLength(1);
expect(fetchMock.calls(DATASOURCE_ENDPOINT)[0][0]).toContain(
'Vehicle%20Sales%20%2B%27',
'Vehicle+Sales%20%2B',
);
fetchMock.reset();
done();