VbzCart/tables/stk places - Revision history

Woozle: one more column: isActivated

2017-04-22T11:56:25Z

one more column: isActivated

Woozle: rewrite of changes

2017-04-22T11:18:55Z

rewrite of changes

← Older revision		Revision as of 11:18, 22 April 2017
Line 5:		Line 5:
	** '''2012-02-06''' added '''isEnabled''' field so cat_items stock can be calculated properly by query		** '''2012-02-06''' added '''isEnabled''' field so cat_items stock can be calculated properly by query
	*** Also corrected type for isActive to match actual table		*** Also corrected type for isActive to match actual table
	** '''2017-04-22''' Finally worked out ~~that this~~ terminology ~~is basically backwards~~. A Place is "active" if it is "enabled" AND is not contained by a Place that has been disabled (un-"enabled").		** '''2017-04-22'''
	*** At first, I thought maybe the caching of "is this Place within a Place that has not been disabled?" was unnecessary, but it turns out the customer-facing calculations of what is currently in stock use queries which depend on the isEnabled field. So ~~the following things need~~ to happen:		*** Finally worked out better terminology. A Place is "active" if it is "enabled" AND is not contained by a Place that has been disabled (un-"enabled") -- but that's kind of the opposite of how we were using the terms. Rather than reverse them, I came up with something more specific and unambiguous.
	**** "isEnabled" should be renamed "isActiveSpace" (done)		**** Renamed "isEnabled" to "isActiveSpace".
	**** "isActive" should be renamed "isSelfActive" (done)		**** Renamed "isActive" to "isSelfActive".
	**** Whenever a Place record is saved and isSelfActive has changed, the code should update "isActiveSpace" for all contained Places.		*** At first, I thought maybe the caching of "is this Place within a Place that has not been disabled?" was unnecessary, but it turns out the customer-facing calculations of what is currently in stock use queries which depend on the isEnabled field. So now this needs to happen:
	*** Also changing from MYISAM to InnoDB, because there's really no reason to use MYISAM. (apparently this was done earlier)		**** Whenever a Place record is saved and '''isSelfActive''' has changed, the code should update '''isActiveSpace''' for all contained Places.
			*** Also changing from MYISAM to InnoDB, because there's really no reason to use MYISAM. (apparently this was done to the actual db earlier)
	* '''Fields''':		* '''Fields''':
	** '''isSelfActive''': if FALSE, then normal operations should ignore this place ''and all places inside it''.		** '''isSelfActive''': if FALSE, then normal operations should ignore this place ''and all places inside it''.
	** '''isActiveSpace''': (calculated ~~by code~~) if TRUE, all parent-container Places are active (isSelfActive = TRUE)		** '''isActiveSpace''': (calculated) if TRUE, all parent-container Places are active (isSelfActive = TRUE)
	==SQL==		==SQL==
	<mysql>CREATE TABLE `stk_places` (		<mysql>CREATE TABLE `stk_places` (

Woozle: flag revisions

2017-04-22T11:14:49Z

flag revisions

← Older revision		Revision as of 11:14, 22 April 2017
Line 5:		Line 5:
	** '''2012-02-06''' added '''isEnabled''' field so cat_items stock can be calculated properly by query		** '''2012-02-06''' added '''isEnabled''' field so cat_items stock can be calculated properly by query
	*** Also corrected type for isActive to match actual table		*** Also corrected type for isActive to match actual table
			** '''2017-04-22''' Finally worked out that this terminology is basically backwards. A Place is "active" if it is "enabled" AND is not contained by a Place that has been disabled (un-"enabled").
			*** At first, I thought maybe the caching of "is this Place within a Place that has not been disabled?" was unnecessary, but it turns out the customer-facing calculations of what is currently in stock use queries which depend on the isEnabled field. So the following things need to happen:
			**** "isEnabled" should be renamed "isActiveSpace" (done)
			**** "isActive" should be renamed "isSelfActive" (done)
			**** Whenever a Place record is saved and isSelfActive has changed, the code should update "isActiveSpace" for all contained Places.
			*** Also changing from MYISAM to InnoDB, because there's really no reason to use MYISAM. (apparently this was done earlier)
	* '''Fields''':		* '''Fields''':
	** '''~~isActive~~''': if FALSE, then normal operations should ignore this place ''and all places inside it''.		** '''isSelfActive''': if FALSE, then normal operations should ignore this place ''and all places inside it''.
	** '''~~isEnabled~~''': (calculated by code) if TRUE, all ~~parents between here and root~~ are ~~Active~~		** '''isActiveSpace''': (calculated by code) if TRUE, all parent-container Places are active (isSelfActive = TRUE)
	==SQL==		==SQL==
	<mysql>CREATE TABLE `stk_places` (		<mysql>CREATE TABLE `stk_places` (
	`ID` INT NOT NULL AUTO_INCREMENT,		`ID` INT NOT NULL AUTO_INCREMENT,
	`ID_Parent` INT COMMENT "self.ID of place in which this place is found; may be NULL",		`ID_Parent` INT COMMENT "self.ID of place in which this place is found; may be NULL",
	`~~isActive~~` TINYINT(1) NOT NULL COMMENT "FALSE = don't normally show this place in lists or count its stock in totals",		`isSelfActive` TINYINT(1) NOT NULL COMMENT "FALSE = don't normally show this place in lists or count its stock in totals",
	`~~isEnabled~~` TINYINT(1) NOT NULL COMMENT "(calculated field) TRUE = all ~~parents between here and root~~ are ~~Active~~",		`isActiveSpace` TINYINT(1) NOT NULL COMMENT "(calculated field) TRUE = all ancestors are isSelfActive=TRUE",
	`Name` VARCHAR(63) NOT NULL COMMENT "brief descriptive name for listings and tree-paths",		`Name` VARCHAR(63) NOT NULL COMMENT "brief descriptive name for listings and tree-paths",
	`Descr` VARCHAR(127) DEFAULT NULL COMMENT "(optional) description of this place, so it can be located",		`Descr` VARCHAR(127) DEFAULT NULL COMMENT "(optional) description of this place, so it can be located",
	PRIMARY KEY(`ID`)		PRIMARY KEY(`ID`)
	) ENGINE = ~~MYISAM~~;</mysql>		) ENGINE = InnoDB;</mysql>
	==Archive==		==Archive==
	These notes do not currently apply. They may be things we will want to use at some point, but are not currently in use on this table.		These notes do not currently apply. They may be things we will want to use at some point, but are not currently in use on this table.

Woozle: link to bins; removed obsolete markup

2015-11-06T21:24:06Z

link to bins; removed obsolete markup

← Older revision		Revision as of 21:24, 6 November 2015
Line 1:		Line 1:
	==About==		==About==
	* '''Purpose''': places where ~~containers~~ may be found, organized hierarchically		* '''Purpose''': places where {{vbzcart\|table\|stk_bins\|bins}} may be found, organized hierarchically
	* '''History''':		* '''History''':
	** '''2010-11-30''' added '''isActive''' field, so we can hide all the old GA places		** '''2010-11-30''' added '''isActive''' field, so we can hide all the old GA places
Line 9:		Line 9:
	** '''isEnabled''': (calculated by code) if TRUE, all parents between here and root are Active		** '''isEnabled''': (calculated by code) if TRUE, all parents between here and root are Active
	==SQL==		==SQL==
	~~<section begin=sql />~~<mysql>CREATE TABLE `stk_places` (		<mysql>CREATE TABLE `stk_places` (
	`ID` INT NOT NULL AUTO_INCREMENT,		`ID` INT NOT NULL AUTO_INCREMENT,
	`ID_Parent` INT COMMENT "self.ID of place in which this place is found; may be NULL",		`ID_Parent` INT COMMENT "self.ID of place in which this place is found; may be NULL",
Line 18:		Line 18:
	PRIMARY KEY(`ID`)		PRIMARY KEY(`ID`)
	) ENGINE = MYISAM;</mysql>		) ENGINE = MYISAM;</mysql>
	~~<section end=sql />~~
	==Archive==		==Archive==
	These notes do not currently apply. They may be things we will want to use at some point, but are not currently in use on this table.		These notes do not currently apply. They may be things we will want to use at some point, but are not currently in use on this table.

Woozle: corrections to match actual table

2012-02-06T15:47:53Z

corrections to match actual table

← Older revision		Revision as of 15:47, 6 February 2012
Line 4:		Line 4:
	** '''2010-11-30''' added '''isActive''' field, so we can hide all the old GA places		** '''2010-11-30''' added '''isActive''' field, so we can hide all the old GA places
	** '''2012-02-06''' added '''isEnabled''' field so cat_items stock can be calculated properly by query		** '''2012-02-06''' added '''isEnabled''' field so cat_items stock can be calculated properly by query
			*** Also corrected type for isActive to match actual table
	* '''Fields''':		* '''Fields''':
	** '''isActive''': if FALSE, then normal operations should ignore this place ''and all places inside it''.		** '''isActive''': if FALSE, then normal operations should ignore this place ''and all places inside it''.
Line 10:		Line 11:
	<section begin=sql /><mysql>CREATE TABLE `stk_places` (		<section begin=sql /><mysql>CREATE TABLE `stk_places` (
	`ID` INT NOT NULL AUTO_INCREMENT,		`ID` INT NOT NULL AUTO_INCREMENT,
	`ID_Parent` INT COMMENT "self.ID of place in which this place is found; may be NULL",		`ID_Parent` INT COMMENT "self.ID of place in which this place is found; may be NULL",
	`isActive` ~~BOOL~~ COMMENT "FALSE = don't normally show this place in lists or count its stock in totals",		`isActive` TINYINT(1) NOT NULL COMMENT "FALSE = don't normally show this place in lists or count its stock in totals",
	`isEnabled` ~~BOOL~~ COMMENT "(calculated field) TRUE = all parents between here and root are Active",		`isEnabled` TINYINT(1) NOT NULL COMMENT "(calculated field) TRUE = all parents between here and root are Active",
	`Name` VARCHAR(63) NOT NULL COMMENT "brief descriptive name for listings and tree-paths",		`Name` VARCHAR(63) NOT NULL COMMENT "brief descriptive name for listings and tree-paths",
	`Descr` VARCHAR(127) COMMENT "(optional) description of this place, so it can be located",		`Descr` VARCHAR(127) DEFAULT NULL COMMENT "(optional) description of this place, so it can be located",
	PRIMARY KEY(`ID`)		PRIMARY KEY(`ID`)
	) ENGINE = MYISAM;</mysql>		) ENGINE = MYISAM;</mysql>

Woozle: isEnabled field

2012-02-06T15:26:09Z

isEnabled field

← Older revision		Revision as of 15:26, 6 February 2012
Line 3:		Line 3:
	* '''History''':		* '''History''':
	** '''2010-11-30''' added '''isActive''' field, so we can hide all the old GA places		** '''2010-11-30''' added '''isActive''' field, so we can hide all the old GA places
			** '''2012-02-06''' added '''isEnabled''' field so cat_items stock can be calculated properly by query
	* '''Fields''':		* '''Fields''':
	** '''isActive''': if FALSE, then normal operations should ignore this place ''and all places inside it''.		** '''isActive''': if FALSE, then normal operations should ignore this place ''and all places inside it''.
			** '''isEnabled''': (calculated by code) if TRUE, all parents between here and root are Active
	==SQL==		==SQL==
	<section begin=sql /><mysql> ~~DROP TABLE IF EXISTS `stk_places`;~~		<section begin=sql /><mysql>CREATE TABLE `stk_places` (

	CREATE TABLE `stk_places` (
	`ID` INT NOT NULL AUTO_INCREMENT,		`ID` INT NOT NULL AUTO_INCREMENT,
	`ID_Parent` INT COMMENT "self.ID of place in which this place is found; may be NULL",		`ID_Parent` INT COMMENT "self.ID of place in which this place is found; may be NULL",
	`isActive` BOOL COMMENT "FALSE = don't normally show this place in lists or count its stock in totals",		`isActive` BOOL COMMENT "FALSE = don't normally show this place in lists or count its stock in totals",
			`isEnabled` BOOL COMMENT "(calculated field) TRUE = all parents between here and root are Active",
	`Name` VARCHAR(63) NOT NULL COMMENT "brief descriptive name for listings and tree-paths",		`Name` VARCHAR(63) NOT NULL COMMENT "brief descriptive name for listings and tree-paths",
	`Descr` VARCHAR(127) COMMENT "(optional) description of this place, so it can be located",		`Descr` VARCHAR(127) COMMENT "(optional) description of this place, so it can be located",

Woozle: +isActive field

2010-11-30T20:14:23Z

+isActive field

← Older revision		Revision as of 20:14, 30 November 2010
Line 1:		Line 1:
	==About==		==About==
	* '''Purpose''': places where containers may be found, organized hierarchically		* '''Purpose''': places where containers may be found, organized hierarchically
	* '''~~Rules~~''': ~~Both stock Bins and order Packages can be in Places. Later~~, ~~Places may have attributes to tell us whether~~ we can ~~ship items directly from there (or at~~ all~~) and whether or not they are effectively "local" to us, but for now they don't.~~		* '''History''':
			** '''2010-11-30''' added '''isActive''' field, so we can hide all the old GA places
	* '''Fields''':		* '''Fields''':
	** '''~~isLocal~~''': FALSE ~~= items aren't where we are~~, ~~so they must either be shipped here first or shipped to customer in separate pkg~~		** '''isActive''': if FALSE, then normal operations should ignore this place ''and all places inside it''.
	** '''isWhse'''~~: TRUE = items have to be fetched for shipping; some delay involved,~~ and ~~may want to print out lists of stuff to fetch~~
	** ''~~'ID_Supp''': NOT NULL = this location is a supplier's stock, and items must be ordered from it in order to be shippable~~
	==SQL==		==SQL==
	<section begin=sql /><mysql> DROP TABLE IF EXISTS `stk_places`;		<section begin=sql /><mysql> DROP TABLE IF EXISTS `stk_places`;

	CREATE TABLE `stk_places` (		CREATE TABLE `stk_places` (
	`ID` INT NOT NULL AUTO_INCREMENT,		`ID` INT NOT NULL AUTO_INCREMENT,
	`ID_Parent` INT COMMENT 'self.ID of place in which this place is found; may be NULL',		`ID_Parent` INT COMMENT "self.ID of place in which this place is found; may be NULL",
	`Name` VARCHAR(63) NOT NULL COMMENT 'brief descriptive name for listings and tree-paths',		`isActive` BOOL COMMENT "FALSE = don't normally show this place in lists or count its stock in totals",
	`Descr` VARCHAR(127) COMMENT '(optional) description of this place, so it can be located',		`Name` VARCHAR(63) NOT NULL COMMENT "brief descriptive name for listings and tree-paths",
			`Descr` VARCHAR(127) COMMENT "(optional) description of this place, so it can be located",
	PRIMARY KEY(`ID`)		PRIMARY KEY(`ID`)
	) ENGINE = MYISAM;</mysql>		) ENGINE = MYISAM;</mysql>
	<section end=sql />		<section end=sql />
			==Archive==
			These notes do not currently apply. They may be things we will want to use at some point, but are not currently in use on this table.
			* '''Rules''': Both stock Bins and order Packages can be in Places. Later, Places may have attributes to tell us whether we can ship items directly from there (or at all) and whether or not they are effectively "local" to us, but for now they don't.
			* '''Fields''':
			** '''isLocal''': FALSE = items aren't where we are, so they must either be shipped here first or shipped to customer in separate pkg
			** '''isWhse''': TRUE = items have to be fetched for shipping; some delay involved, and may want to print out lists of stuff to fetch
			** '''ID_Supp''': NOT NULL = this location is a supplier's stock, and items must be ordered from it in order to be shippable

Woozle: extracted from VbzCart/tables

2008-12-29T17:41:09Z

extracted from VbzCart/tables

New page

==About==
* '''Purpose''': places where containers may be found, organized hierarchically
* '''Rules''': Both stock Bins and order Packages can be in Places. Later, Places may have attributes to tell us whether we can ship items directly from there (or at all) and whether or not they are effectively "local" to us, but for now they don't.
* '''Fields''':
** '''isLocal''': FALSE = items aren't where we are, so they must either be shipped here first or shipped to customer in separate pkg
** '''isWhse''': TRUE = items have to be fetched for shipping; some delay involved, and may want to print out lists of stuff to fetch
** '''ID_Supp''': NOT NULL = this location is a supplier's stock, and items must be ordered from it in order to be shippable
==SQL==
<section begin=sql /><mysql> DROP TABLE IF EXISTS `stk_places`;

CREATE TABLE `stk_places` (
`ID` INT NOT NULL AUTO_INCREMENT,
`ID_Parent` INT COMMENT 'self.ID of place in which this place is found; may be NULL',
`Name` VARCHAR(63) NOT NULL COMMENT 'brief descriptive name for listings and tree-paths',
`Descr` VARCHAR(127) COMMENT '(optional) description of this place, so it can be located',
PRIMARY KEY(`ID`)
) ENGINE = MYISAM;</mysql>
<section end=sql />

← Older revision		Revision as of 11:56, 22 April 2017
Line 9:		Line 9:
	**** Renamed "isEnabled" to "isActiveSpace".		**** Renamed "isEnabled" to "isActiveSpace".
	**** Renamed "isActive" to "isSelfActive".		**** Renamed "isActive" to "isSelfActive".
			**** Created "isActivated".
	*** At first, I thought maybe the caching of "is this Place within a Place that has not been disabled?" was unnecessary, but it turns out the customer-facing calculations of what is currently in stock use queries which depend on the isEnabled field. So now this needs to happen:		*** At first, I thought maybe the caching of "is this Place within a Place that has not been disabled?" was unnecessary, but it turns out the customer-facing calculations of what is currently in stock use queries which depend on the isEnabled field. So now this needs to happen:
	**** Whenever a Place record is saved and '''isSelfActive''' has changed, the code should update '''isActiveSpace''' for all contained Places.		**** Whenever a Place record is saved and '''isSelfActive''' has changed, the code should update '''isActiveSpace''' for all contained Places.
Line 15:		Line 16:
	** '''isSelfActive''': if FALSE, then normal operations should ignore this place ''and all places inside it''.		** '''isSelfActive''': if FALSE, then normal operations should ignore this place ''and all places inside it''.
	** '''isActiveSpace''': (calculated) if TRUE, all parent-container Places are active (isSelfActive = TRUE)		** '''isActiveSpace''': (calculated) if TRUE, all parent-container Places are active (isSelfActive = TRUE)
			** '''isActivated''': (calculated): isSelfActive && isActiveSpace -- this is the flag which stock calculations should look at
	==SQL==		==SQL==
	<mysql>CREATE TABLE `stk_places` (		<mysql>CREATE TABLE `stk_places` (
Line 20:		Line 22:
	`ID_Parent` INT COMMENT "self.ID of place in which this place is found; may be NULL",		`ID_Parent` INT COMMENT "self.ID of place in which this place is found; may be NULL",
	`isSelfActive` TINYINT(1) NOT NULL COMMENT "FALSE = don't normally show this place in lists or count its stock in totals",		`isSelfActive` TINYINT(1) NOT NULL COMMENT "FALSE = don't normally show this place in lists or count its stock in totals",
	`isActiveSpace` TINYINT(1) NOT NULL COMMENT "(calculated ~~field~~) TRUE = all ancestors are isSelfActive=TRUE",		`isActiveSpace` TINYINT(1) NOT NULL COMMENT "(calculated) TRUE = all ancestors are isSelfActive=TRUE",
			`isActivated` TINYINT(1) NOT NULL COMMENT "(calculated) TRUE = this Place is active",
	`Name` VARCHAR(63) NOT NULL COMMENT "brief descriptive name for listings and tree-paths",		`Name` VARCHAR(63) NOT NULL COMMENT "brief descriptive name for listings and tree-paths",
	`Descr` VARCHAR(127) DEFAULT NULL COMMENT "(optional) description of this place, so it can be located",		`Descr` VARCHAR(127) DEFAULT NULL COMMENT "(optional) description of this place, so it can be located",