0019456: [P&E] Move hardcoded labels to translatable messages

Notes
(0044216) hgbot (developer) 2012-01-10 18:51	Repository: erp/devel/pi Changeset: 22483d3d1d7fd5bb7c9d2b96d3d71ae3ddc4321b Author: Iván Perdomo <ivan.perdomo <at> openbravo.com> Date: Tue Jan 10 18:49:33 2012 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/22483d3d1d7fd5bb7c9d2b96d3d71ae3ddc4321b [^] Fixes issue 19456: Removed hardcoded label for Done button --- M modules/org.openbravo.client.application/src-db/database/sourcedata/AD_MESSAGE.xml M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/process/ob-pick-and-execute-view.js ---

(0044219) dbaz (developer) 2012-01-10 20:13	The AD_MESSAGE definition is too generic. Two different solutions here: a) Change MSGTIP by something more generic: "Label for 'Done' type buttons" With this change, the label is supposed to be reused in more places. b) Change VALUE by something more explicit, for example, "OBUIAPP_PNE_DoneButton" IMHO I think it is better use option a)

(0044224) iperdomo (reporter) 2012-01-11 07:54	@dbaz: Message tip is just a hint where is used, not every place where is actually used. The initial intention (current) is to use the label "Done" in the Pick-and-Execute processes, but not restricted to that. The JavaScript API to get that label doesn't restrict where to use it. Both proposed solutions are not correct. a) Since is could be used for anything with a label "Done", not only buttons. b) The second proposal is not the usual approach. We use the dbprefix for doing messages "namespacing", but we don't make a 2nd level namespace for an specific scenario

(0044249) dbaz (developer) 2012-01-11 11:56	@iperdomo: Of course that could be used anywhere and of course there are no rules for the name. I said button because I can not imagine an scenario with just the "Done" word besides a button. But of course it could be. My point is that you have used a very abstract VALUE for a very pasticular MSGTIP. Are There rules to quantify how abstract or not should be these things? No. Is there a benefit in writting MSGTIP more abstract? Just MSGTIP coherence if someone else uses the 'Done' label reusing your OBUIAPP_Done

(0044256) iperdomo (reporter) 2012-01-11 13:40	Here there is the other hardcoded label: https://code.openbravo.com/erp/devel/pi-editablepaymentplan/rev/122d77bbfd2a [^]

(0044365) hudsonbot (developer) 2012-01-14 21:21	A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/0f91795528e4 [^] Maturity status: Test

(0045120) dbaz (developer) 2012-02-14 17:15	https://code.openbravo.com/erp/devel/pi/rev/0d5c7c2a2352 [^]

(0045156) AugustoMauch (manager) 2012-02-15 17:06	Code reviewed. The label "Done" and "Add new" are no longer hardcoded.

Issue History
Date Modified	Username	Field	Change
2012-01-10 18:43	iperdomo	New Issue
2012-01-10 18:43	iperdomo	Assigned To	=> iperdomo
2012-01-10 18:43	iperdomo	Web browser	=> Google Chrome
2012-01-10 18:43	iperdomo	Modules	=> Core
2012-01-10 18:46	iperdomo	Status	new => scheduled
2012-01-10 18:46	iperdomo	fix_in_branch	=> pi
2012-01-10 18:51	hgbot	Checkin
2012-01-10 18:51	hgbot	Note Added: 0044216
2012-01-10 18:51	hgbot	Status	scheduled => resolved
2012-01-10 18:51	hgbot	Resolution	open => fixed
2012-01-10 18:51	hgbot	Fixed in SCM revision	=> http://code.openbravo.com/erp/devel/pi/rev/22483d3d1d7fd5bb7c9d2b96d3d71ae3ddc4321b [^]
2012-01-10 20:13	dbaz	Note Added: 0044219
2012-01-11 07:54	iperdomo	Note Added: 0044224
2012-01-11 11:56	dbaz	Note Added: 0044249
2012-01-11 13:32	iperdomo	Status	resolved => new
2012-01-11 13:32	iperdomo	Resolution	fixed => open
2012-01-11 13:37	iperdomo	Web browser	Google Chrome => Google Chrome
2012-01-11 13:37	iperdomo	fix_in_branch	pi =>
2012-01-11 13:37	iperdomo	Summary	[P&E] Move hardcoded label "Done" to a Message => [P&E] Move hardcoded labels to translatable messages
2012-01-11 13:37	iperdomo	Description Updated	View Revisions
2012-01-11 13:37	iperdomo	Steps to Reproduce Updated	View Revisions
2012-01-11 13:40	iperdomo	Note Added: 0044256
2012-01-14 21:21	hudsonbot	Checkin
2012-01-14 21:21	hudsonbot	Note Added: 0044365
2012-01-30 12:10	iperdomo	Web browser	Google Chrome => Google Chrome
2012-01-30 12:10	iperdomo	Target Version	3.0MP8 => 3.0MP9
2012-02-08 18:59	dbaz	Assigned To	iperdomo => dbaz
2012-02-14 17:15	dbaz	Status	new => scheduled
2012-02-14 17:15	dbaz	fix_in_branch	=> pi
2012-02-14 17:15	dbaz	Note Added: 0045120
2012-02-14 17:15	dbaz	Status	scheduled => resolved
2012-02-14 17:15	dbaz	Fixed in Version	=> 3.0MP9
2012-02-14 17:15	dbaz	Resolution	open => fixed
2012-02-15 17:06	AugustoMauch	Note Added: 0045156
2012-02-15 17:06	AugustoMauch	Status	resolved => closed

Relationships [ Relation Graph ] [ Dependency Graph ]