Skip to content

Commit 98f692f

Browse files
authored
Merge pull request #51 from lizlooney/pr_fix_python_regeneration
Fixed code that triggers python regeneration.
2 parents 38872c1 + 532c663 commit 98f692f

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

src/App.tsx

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ const App: React.FC = () => {
238238
const [alertErrorVisible, setAlertErrorVisible] = useState(false);
239239
const [mostRecentModulePath, setMostRecentModulePath] = useState<string | null>(null);
240240
const [shownPythonToolboxCategories, setShownPythonToolboxCategories] = useState<Set<string>>(new Set());
241-
const [triggerListModules, setTriggerListModules] = useState(false);
241+
const [triggerListModules, setTriggerListModules] = useState(0);
242242
const afterListModulesSuccess = useRef<() => void>(() => {});
243243
const [modules, setModules] = useState<commonStorage.Project[]>([]);
244244
const [treeData, setTreeData] = useState<TreeDataNode[]>([]);
@@ -250,7 +250,7 @@ const App: React.FC = () => {
250250
const [copyTooltip, setCopyTooltip] = useState('Copy');
251251
const [deleteTooltip, setDeleteTooltip] = useState('Delete');
252252
const blocklyComponent = useRef<BlocklyComponentType | null>(null);
253-
const [triggerPythonRegeneration, setTriggerPythonRegeneration] = useState(false);
253+
const [triggerPythonRegeneration, setTriggerPythonRegeneration] = useState(0);
254254
const blocksEditor = useRef<editor.Editor | null>(null);
255255
const [generatedCode, setGeneratedCode] = useState('');
256256
const [newProjectNameModalPurpose, setNewProjectNameModalPurpose] = useState('');
@@ -339,7 +339,7 @@ const App: React.FC = () => {
339339
// mostRecentModulePath hasn't been fetched yet. Try agagin in a bit.
340340
if (mostRecentModulePath == null) {
341341
setTimeout(() => {
342-
setTriggerListModules(!triggerListModules);
342+
setTriggerListModules(Date.now());
343343
}, 50);
344344
return;
345345
}
@@ -537,7 +537,7 @@ const App: React.FC = () => {
537537
// Don't regenerate python code mid-drag.
538538
return;
539539
}
540-
setTriggerPythonRegeneration(!triggerPythonRegeneration);
540+
setTriggerPythonRegeneration(Date.now());
541541
};
542542

543543
const handlePopconfirmOk = () => {
@@ -603,7 +603,7 @@ const App: React.FC = () => {
603603
afterListModulesSuccess.current = () => {
604604
setCurrentModulePath(newProjectPath);
605605
};
606-
setTriggerListModules(!triggerListModules);
606+
setTriggerListModules(Date.now());
607607
} else if (errorMessage) {
608608
setAlertErrorMessage('Failed to create a new Project: ' + errorMessage);
609609
setAlertErrorVisible(true);
@@ -618,7 +618,7 @@ const App: React.FC = () => {
618618
afterListModulesSuccess.current = () => {
619619
setCurrentModulePath(newProjectPath);
620620
};
621-
setTriggerListModules(!triggerListModules);
621+
setTriggerListModules(Date.now());
622622
} else if (errorMessage) {
623623
setAlertErrorMessage('Failed to rename the Project: ' + errorMessage);
624624
setAlertErrorVisible(true);
@@ -633,7 +633,7 @@ const App: React.FC = () => {
633633
afterListModulesSuccess.current = () => {
634634
setCurrentModulePath(newProjectPath);
635635
};
636-
setTriggerListModules(!triggerListModules);
636+
setTriggerListModules(Date.now());
637637
} else if (errorMessage) {
638638
setAlertErrorMessage('Failed to copy the Project: ' + errorMessage);
639639
setAlertErrorVisible(true);
@@ -695,7 +695,7 @@ const App: React.FC = () => {
695695
afterListModulesSuccess.current = () => {
696696
setCurrentModulePath(newModulePath);
697697
};
698-
setTriggerListModules(!triggerListModules);
698+
setTriggerListModules(Date.now());
699699
} else if (errorMessage) {
700700
setAlertErrorMessage('Failed to create a new Mechanism: ' + errorMessage);
701701
setAlertErrorVisible(true);
@@ -710,7 +710,7 @@ const App: React.FC = () => {
710710
afterListModulesSuccess.current = () => {
711711
setCurrentModulePath(newModulePath);
712712
};
713-
setTriggerListModules(!triggerListModules);
713+
setTriggerListModules(Date.now());
714714
} else if (errorMessage) {
715715
setAlertErrorMessage('Failed to create a new OpMode: ' + errorMessage);
716716
setAlertErrorVisible(true);
@@ -725,7 +725,7 @@ const App: React.FC = () => {
725725
afterListModulesSuccess.current = () => {
726726
setCurrentModulePath(newModulePath);
727727
};
728-
setTriggerListModules(!triggerListModules);
728+
setTriggerListModules(Date.now());
729729
} else if (errorMessage) {
730730
setAlertErrorMessage('Failed to rename the module: ' + errorMessage);
731731
setAlertErrorVisible(true);
@@ -740,7 +740,7 @@ const App: React.FC = () => {
740740
afterListModulesSuccess.current = () => {
741741
setCurrentModulePath(newModulePath);
742742
};
743-
setTriggerListModules(!triggerListModules);
743+
setTriggerListModules(Date.now());
744744
} else if (errorMessage) {
745745
setAlertErrorMessage('Failed to copy the module: ' + errorMessage);
746746
setAlertErrorVisible(true);
@@ -864,7 +864,7 @@ const App: React.FC = () => {
864864
storage.deleteModule(moduleTypeToDelete, modulePathToDelete,
865865
(success: boolean, errorMessage: string) => {
866866
if (success) {
867-
setTriggerListModules(!triggerListModules);
867+
setTriggerListModules(Date.now());
868868
} else if (errorMessage) {
869869
setAlertErrorMessage('Failed to delete the Project: ' + errorMessage);
870870
setAlertErrorVisible(true);
@@ -882,7 +882,7 @@ const App: React.FC = () => {
882882
storage.deleteModule(moduleTypeToDelete, modulePathToDelete,
883883
(success: boolean, errorMessage: string) => {
884884
if (success) {
885-
setTriggerListModules(!triggerListModules);
885+
setTriggerListModules(Date.now());
886886
} else if (errorMessage) {
887887
setAlertErrorMessage('Failed to delete the module: ' + errorMessage);
888888
setAlertErrorVisible(true);

0 commit comments

Comments
 (0)