minor changs to delete app #221
Conversation
There was a problem hiding this comment.
aiagallery.dbif.ObjSearch does not have a method deleteApp(), unless you added it, and it seems inappropriate. What was the reason for this change?
There was a problem hiding this comment.
You said that I need to change removeSelf so that the app is not removed just marked as deleted. I did not get to implement it in objSearch.
Sent from my iPhone
On Dec 5, 2011, at 9:52 AM, Derrell Lipman reply@reply.github.com wrote:
@@ -221,7 +221,7 @@ qx.Mixin.define("aiagallery.dbif.MApps",
searchObj = new aiagallery.dbif.ObjSearch([obj["word"],
obj["appId"],
obj["appField"]]);
searchObj.removeSelf();searchObj.deleteApp();aiagallery.dbif.ObjSearch does not have a method deleteApp(), unless you added it, and it seems inappropriate. What was the reason for this change?
Reply to this email directly or view it on GitHub:
https://github.com/derrell/App-Inventor-Gallery/pull/221/files#r269870
There was a problem hiding this comment.
On Mon, Dec 5, 2011 at 10:14, piersquared <
reply@reply.github.com
wrote:
@@ -221,7 +221,7 @@ qx.Mixin.define("aiagallery.dbif.MApps",
searchObj = new
aiagallery.dbif.ObjSearch([obj["word"],
obj["appId"],
obj["appField"]]);
searchObj.removeSelf();searchObj.deleteApp();You said that I need to change removeSelf so that the app is not removed
just marked as deleted. I did not get to implement it in objSearch.
Oh, ok, I see. That's not something that goes in ObjSearch, however. That's
code that goes right where you inserted the deleteApp() call, to change its
data. (The Obj* classes are all identically organized, with a constructor
and a defer function, but no application-specific member methods.)
Derrell
2 participants
No description provided.