derrell/App-Inventor-Gallery — Issue #172: Appinfo module needs restructuring.
Author: mijoda
Created: 2011-11-07 04:32:27+00:00
State: open
Labels: SWE

The 3 classes in aiagallery.module.dgallery.appinfo are singletons, despite the fact that there should be multiple instances when multiple appinfo tabs are open. One reason this module has been working is that certain variables referenced in different parts of Gui.js have been passed through the Fsm. Attempting to convert these into instance variables revealed this issue (which probably should have been obvious earlier). Liking and commenting in one app tab would affect a different one; amusing, but very broken.

Derrell changed AppInfo.js to not be a singleton (commit f88499d, 10/28). I need to do the same for Gui.js and Fsm.js by, at least, removing the 'type : "singleton"' lines, and replacing all getInstance()'s with the appropriate incantation.