Address useRef in TableResultsEntities component

[tjmadonna]

I get occasional errors while running the portal locally. I believe it's caused by the conditional ref.

portal-ui/src/components/custom/TableResultsEntities.jsx

Line 27 in dd3606c

currentColumns = currentColumns || useRef([])

[libpitt]

Hmm. Yes. So the collections/Datasets.jsx needed to have its own ref. A ref was used in the first place instead of regular JS array for value persistence.

Anyhow a possible fix for this would be to remove the condition.

function TableResultsEntities({children, filters, onRowClicked, currentColumns,...}) {
:
const _currentColumns = useRef([])
:

// Then have:
setCurrentColumns = (cols) => {
	if (currentColumns) {
		currentColumns.current = cols
	} else {
		_currentColumns.current = cols
	}
}

getCurrentColumns = () => currentColumns.current || _currentColumns.current

But I cannot reproduce the problem. Don't want to implement a blind solution.

I do think the whole table has gotten out of hand with all the customizations. Wish antd.Table was known before it all.

@tjmadonna