Talk:Backgrounds tutorial - Revision history

FollowTheGourd: /* Bugfixes and some comments */

2011-03-29T03:28:08Z

‎Bugfixes and some comments

FollowTheGourd: /* what's the preview button do? */

2011-03-29T03:27:26Z

‎what's the preview button do?

FollowTheGourd: /* Bugfixes and some comments */

2011-03-29T03:26:40Z

‎Bugfixes and some comments

FollowTheGourd: /* When is an M2DA not like an M2DA? */

2011-03-29T03:25:23Z

‎When is an M2DA not like an M2DA?

FollowTheGourd: /* When is an M2DA not like an M2DA? */

2011-03-29T03:25:04Z

‎When is an M2DA not like an M2DA?

FollowTheGourd: /* Bugfixes and some comments */

2011-03-29T03:24:01Z

‎Bugfixes and some comments

FollowTheGourd: /* Bugfixes and some comments */

2011-03-29T03:23:00Z

‎Bugfixes and some comments

FollowTheGourd: /And forgot again /

2011-03-29T02:44:01Z

‎And forgot again

FollowTheGourd: /* oops */

2011-03-29T02:42:08Z

‎oops

FollowTheGourd: Bug fix for crashing because of higher string IDs

2011-03-29T02:37:53Z

Bug fix for crashing because of higher string IDs

← Older revision		Revision as of 03:28, 29 March 2011
Line 4:		Line 4:

	The crashing because of the background description string ID is actually because of the UI and can be fixed by modders. In the chargen.gfx's ActionScript RaceGenderScene.GrabBackgrounds() static function, the following is what's causing the crash - basically using the StringID as an array index for the sole purpose of checking if that StringID has been seen yet, so this can be worked around to check it some way else - or just drop it if it's not strictly needed (Apparently it is, but see below the code). Incidentally, this is probably also why you need unique background description string IDs for each race for them to get properly enumerated.		The crashing because of the background description string ID is actually because of the UI and can be fixed by modders. In the chargen.gfx's ActionScript RaceGenderScene.GrabBackgrounds() static function, the following is what's causing the crash - basically using the StringID as an array index for the sole purpose of checking if that StringID has been seen yet, so this can be worked around to check it some way else - or just drop it if it's not strictly needed (Apparently it is, but see below the code). Incidentally, this is probably also why you need unique background description string IDs for each race for them to get properly enumerated.
		+
		+	You could just constrain your string ID to a lower value given the other mentioned workaround, but wouldn't it be nice not to have to worry about bugs?

	'''This is the code causing the crash:'''		'''This is the code causing the crash:'''

@@ Line 5: / Line 5: @@
 The crashing because of the background description string ID is actually because of the UI and can be fixed by modders. In the chargen.gfx's ActionScript RaceGenderScene.GrabBackgrounds() static function, the following is what's causing the crash - basically using the StringID as an array index for the sole purpose of checking if that StringID has been seen yet, so this can be worked around to check it some way else - or just drop it if it's not strictly needed (Apparently it is, but see below the code). Incidentally, this is probably also why you need unique background description string IDs for each race for them to get properly enumerated.
-'''This is the code causing the crash'''
+'''This is the code causing the crash:'''
 <dascript>
 var aStringCheck = [];
@@ Line 21: / Line 21: @@
 The check *is* needed or else the background will show up active in the list of each race that possesses it. One simple workaround is use a dictionary instead of an array:
 '''This would seem to be a workaround'''
 <dascript>
 var aStringCheck = {};

@@ Line 4: / Line 4: @@
 The crashing because of the background description string ID is actually because of the UI and can be fixed by modders. In the chargen.gfx's ActionScript RaceGenderScene.GrabBackgrounds() static function, the following is what's causing the crash - basically using the StringID as an array index for the sole purpose of checking if that StringID has been seen yet, so this can be worked around to check it some way else - or just drop it if it's not strictly needed (Apparently it is, but see below the code). Incidentally, this is probably also why you need unique background description string IDs for each race for them to get properly enumerated.
 <dascript>
 var aStringCheck = [];
@@ Line 19: / Line 21: @@
 The check *is* needed or else the background will show up active in the list of each race that possesses it. One simple workaround is use a dictionary instead of an array:
 <dascript>
 var aStringCheck = {};

← Older revision		Revision as of 03:25, 29 March 2011
Line 47:		Line 47:

	Of course if there's a way to derive the row ID (by a prearranged formula or given by another M2DA cell's value), then the UI can use it just like a normal M2DA.		Of course if there's a way to derive the row ID (by a prearranged formula or given by another M2DA cell's value), then the UI can use it just like a normal M2DA.
		+
		+	--[[User:FollowTheGourd\|FollowTheGourd]] 03:25, 29 March 2011 (UTC)

	== Is this correct? ==		== Is this correct? ==

@@ Line 46: / Line 46: @@
 Data
+Of course if there's a way to derive the row ID (by a prearranged formula or given by another M2DA cell's value), then the UI can use it just like a normal M2DA.
-−
 Of course if there's a way to derive the row ID, then the UI can use it just like a normal M2DA.
 == Is this correct? ==

@@ Line 7: / Line 7: @@
 var aStringCheck = [];
 // ...
-// ... nested 'for-loop's
+// ... nested for-loops
         var bBackgroundAllowedByRace = Number(ExternalCommands.Get2DAValueByRowIndexColumnName(_2DA.TABLE_BACKGROUND, nBackgroundRow, RaceGenderScene.m_aRaces[nRaceIndex].sLabel));
         var nDescStringID = Number(ExternalCommands.Get2DAValueByRowIndexColumnName(_2DA.TABLE_BACKGROUND_DESC, nBackgroundRow, RaceGenderScene.m_aRaces[nRaceIndex].sLabel));
@@ Line 15: / Line 15: @@
+        }
-// ... end nested 'for-loop's
+// ... end nested for-loops
 </dascript>
 The check *is* needed or else the background will show up active in the list of each race that possesses it. One simple workaround is use a dictionary instead of an array:
 var aStringCheck = {};
 // ...
-// ... nested 'for-loop's
+// ... nested for-loops
         var bBackgroundAllowedByRace = Number(ExternalCommands.Get2DAValueByRowIndexColumnName(_2DA.TABLE_BACKGROUND, nBackgroundRow, RaceGenderScene.m_aRaces[nRaceIndex].sLabel));
         var nDescStringID = Number(ExternalCommands.Get2DAValueByRowIndexColumnName(_2DA.TABLE_BACKGROUND_DESC, nBackgroundRow, RaceGenderScene.m_aRaces[nRaceIndex].sLabel));
@@ Line 29: / Line 30: @@
+        }
-// ... end nested 'for-loop's
+// ... end nested for-loops
 </dascript>

← Older revision		Revision as of 02:37, 29 March 2011
Line 1:		Line 1:
		+	__TOC__
		+
		+	== Bugfixes and some comments ==
		+
		+	The crashing because of the background description string ID is actually because of the UI and can be fixed by modders. In the chargen.gfx's ActionScript RaceGenderScene.GrabBackgrounds() static function, the following is what's causing the crash - basically using the StringID as an array index for the sole purpose of checking if that StringID has been seen yet, so this can be worked around to check it some way else - or just drop it if it's not strictly needed. Incidentally, this is probably also why you need unique background description string IDs for each race for them to get properly enumerated.
		+	<dascript>
		+	var = aStringCheck[];
		+	// ...
		+	// ... nested 'for-loop's
		+	var bBackgroundAllowedByRace = Number(ExternalCommands.Get2DAValueByRowIndexColumnName(_2DA.TABLE_BACKGROUND, nBackgroundRow, RaceGenderScene.m_aRaces[nRaceIndex].sLabel));
		+	var nDescStringID = Number(ExternalCommands.Get2DAValueByRowIndexColumnName(_2DA.TABLE_BACKGROUND_DESC, nBackgroundRow, RaceGenderScene.m_aRaces[nRaceIndex].sLabel));
		+	if (bBackgroundAllowedByRace == 1 && aStringCheck[nDescStringID] == null)
		+	{
		+	aStringCheck[nDescStringID] = nDescStringID;
		+
		+	}
		+	// ... end nested 'for-loop's
		+	</dascript>
		+
	== Is this correct? ==		== Is this correct? ==

Talk:Backgrounds tutorial - Revision history

FollowTheGourd: /* Bugfixes and some comments */

FollowTheGourd: /* what's the preview button do? */

FollowTheGourd: /* Bugfixes and some comments */

FollowTheGourd: /* When is an M2DA not like an M2DA? */

FollowTheGourd: /* When is an M2DA not like an M2DA? */

FollowTheGourd: /* Bugfixes and some comments */

FollowTheGourd: /* Bugfixes and some comments */

FollowTheGourd: /*And forgot again */

FollowTheGourd: /* oops */

FollowTheGourd: Bug fix for crashing because of higher string IDs

FollowTheGourd: /And forgot again /