| [00:05:15] |
*** |
njero has joined #openmrs |
| [00:05:15] |
*** |
ChanServ sets mode: +v njero |
| [00:11:38] |
*** |
njero has quit IRC |
| [00:12:12] |
*** |
njero has joined #openmrs |
| [00:12:12] |
*** |
ChanServ sets mode: +v njero |
| [00:22:10] |
*** |
r0bby|vista has joined #openmrs |
| [00:53:25] |
*** |
r0bby|vista has quit IRC |
| [01:06:33] |
*** |
njero_ has joined #openmrs |
| [01:09:19] |
*** |
njero has quit IRC |
| [02:02:12] |
*** |
njero_ is now known as njero |
| [02:02:44] |
*** |
ChanServ sets mode: +v njero |
| [02:34:09] |
*** |
sioraiocht has quit IRC |
| [02:39:58] |
*** |
Keelhaul has quit IRC |
| [03:43:05] |
*** |
sioraiocht has joined #openmrs |
| [03:43:05] |
*** |
ChanServ sets mode: +v sioraiocht |
| [04:32:38] |
*** |
bmckown has quit IRC |
| [06:39:04] |
*** |
upul has quit IRC |
| [06:45:43] |
*** |
james_regen has joined #openmrs |
| [06:45:43] |
*** |
ChanServ sets mode: +v james_regen |
| [07:03:39] |
*** |
bwolfe has joined #openmrs |
| [07:03:39] |
*** |
ChanServ sets mode: +o bwolfe |
| [08:04:57] |
*** |
sprasanna has joined #openmrs |
| [08:59:57] |
<sprasanna> |
bwolfe: Hi Ben |
| [09:00:45] |
<bwolfe> |
hey sri |
| [09:00:54] |
<sprasanna> |
bwolfe: i see this line "<c:forEach items="${model.layoutTemplate.lines}" var="line">" in addressLayout.jsp but i want to know where the feed comes from... can you help me with that? |
| [09:02:01] |
<bwolfe> |
what do you mean by "feed" ? |
| [09:03:23] |
<sprasanna> |
the jsp file pulls the address component from somewhere to build the <input> tags for the page na... i need to know where it gets those data.. |
| [09:05:37] |
<bwolfe> |
the object returned by model.getLayoutTemplate is an AddressTemplate object |
| [09:05:45] |
<bwolfe> |
AddressTemplate extends LayoutTemplate |
| [09:05:54] |
<bwolfe> |
LayoutTemplate has a method called getLines() |
| [09:06:15] |
<bwolfe> |
getLines() returns the extremely ugly "List<List<Map<String,String>>>" !! |
| [09:06:47] |
<sprasanna> |
oh thanks a lot ben |
| [09:22:48] |
*** |
pearlbear has joined #openmrs |
| [09:22:48] |
*** |
ChanServ sets mode: +v pearlbear |
| [09:53:36] |
*** |
upul has joined #openmrs |
| [10:03:00] |
*** |
sioraiocht has quit IRC |
| [10:17:49] |
<upul> |
Hi bwolfe |
| [10:18:19] |
<bwolfe> |
hey upul |
| [10:19:19] |
<upul> |
it looks like table names are hardcoded in the source |
| [10:21:34] |
<upul> |
about converting sql diff to xml format, problem will be not being able to test with each version |
| [10:22:49] |
<bwolfe> |
upul: what do you mean by test with each version ? |
| [10:23:39] |
<upul> |
convesion is possible, but can't check for mistakes with each version |
| [10:23:59] |
<bwolfe> |
upul: hmm, I still don't know what you mean |
| [10:24:01] |
<upul> |
when adding a sql diff entry that has been tested at that time by the added person |
| [10:24:29] |
<upul> |
but we are not testing each one with each incremental version |
| [10:25:23] |
<bwolfe> |
upul: I don't think converting to the xml is really worth it. if there was a way to call that file from liquibase as the first entry in the new liquibase format, that would work. it would give a common starting point for all future updates. |
| [10:26:28] |
<bwolfe> |
upul: need to run. be back later |
| [10:26:32] |
<upul> |
ok |
| [10:28:03] |
*** |
nribeka has quit IRC |
| [10:30:02] |
*** |
njero_ has joined #openmrs |
| [10:37:24] |
*** |
njero has quit IRC |
| [10:41:04] |
*** |
TorLye has joined #openmrs |
| [10:43:03] |
*** |
njero has joined #openmrs |
| [10:43:03] |
*** |
ChanServ sets mode: +v njero |
| [10:43:46] |
*** |
njero_ has quit IRC |
| [11:01:36] |
*** |
njero has quit IRC |
| [11:01:40] |
*** |
njero_ has joined #openmrs |
| [11:14:49] |
*** |
njero_ has quit IRC |
| [11:15:02] |
*** |
bmckown has joined #openmrs |
| [11:15:02] |
*** |
ChanServ sets mode: +o bmckown |
| [11:15:10] |
*** |
njero has joined #openmrs |
| [11:15:11] |
*** |
ChanServ sets mode: +v njero |
| [11:17:57] |
*** |
TorLye_ has joined #openmrs |
| [11:19:27] |
*** |
TorLye has quit IRC |
| [11:42:24] |
<OpenMRSBot> |
Recent updates in the world of openmrs: Upul Godage: OpenMRS update <http://upulgsoc.wordpress.com/2008/08/12/openmrs-update/> |
| [11:46:34] |
*** |
TorLye_ has quit IRC |
| [11:46:55] |
*** |
TorLye has joined #openmrs |
| [11:49:35] |
<sprasanna> |
bwolfe: y i dont find LayoutTemplate in javadoc api? |
| [11:50:35] |
<bwolfe> |
sprasanna: I think its in the web layer |
| [11:50:46] |
<bwolfe> |
sprasanna: because its an interface/presentatino kind of thing |
| [11:50:54] |
<bwolfe> |
sprasanna: are you using eclipse ? |
| [11:51:01] |
<sprasanna> |
bwolfe: yeah |
| [11:51:12] |
<sprasanna> |
bwolfe: using eclipse |
| [11:51:23] |
<bwolfe> |
sprasanna: if so, you can use control-shift-t, and then type in LayoutTemplate...or even just type in LaTem and you'll see it |
| [11:51:31] |
*** |
njero_ has joined #openmrs |
| [11:52:19] |
<sprasanna> |
bwolfe: its not an interface i guess |
| [11:52:27] |
*** |
njero has quit IRC |
| [11:52:36] |
<sprasanna> |
abstract class |
| [11:52:50] |
<bwolfe> |
yes, abstract class that AddressTemplate and NameTempalte extend |
| [11:53:42] |
<sprasanna> |
if i create a package like org.openmrs.layout.web with a class LayoutTemplate. will it override the existing one?? |
| [11:54:02] |
<sprasanna> |
bwolfe: i tried but it dint work |
| [11:54:20] |
<sprasanna> |
bwolfe: i just wanna confirm whether i am wrong at some place |
| [11:55:00] |
<bwolfe> |
sprasanna: why do you want to override it ? |
| [11:55:07] |
<bwolfe> |
sprasanna: why not just use it ? |
| [11:56:56] |
<sprasanna> |
bwolfe: hmmm.. i just want to alter that getLines() method alone.. if i do that then i need not modify much in controller and addresslayout jsp |
| [11:57:36] |
<bwolfe> |
sprasanna: I see |
| [11:57:47] |
<bwolfe> |
well, you can't "override" a java class like that unfortuantely |
| [11:58:06] |
<sprasanna> |
bwolfe: yeah i know that but gave it a try |
| [11:58:19] |
<bwolfe> |
you might be able to set the template to use to your module class..which just extends TemplateLayout and overrides the getLines method |
| [11:58:49] |
*** |
njero_ has quit IRC |
| [11:58:49] |
<bwolfe> |
sprasanna: the spring bean defines the class to use: <bean id="addressTemplateRwanda" class="org.openmrs.layout.web.address.AddressTemplate"> |
| [11:58:59] |
<bwolfe> |
sprasanna: are you defining your own address layout beans ? |
| [11:59:26] |
<sprasanna> |
bwolfe: not yet but it seems i have to |
| [11:59:58] |
<sprasanna> |
the line you just gave is it from rwanda registration module? |
| [12:00:45] |
<bwolfe> |
sprasanna: its in openmrs-servlet.xml |
| [12:00:55] |
<sprasanna> |
oh |
| [12:03:06] |
<bwolfe> |
sprasanna: ideally those address and name beans in openmrs-servlet.xml are moved to global properties so they can be definied by the user....similar to what darius did with the regimens: http://dev.openmrs.org/ticket/702 |
| [12:04:41] |
<bwolfe> |
brb |
| [12:04:43] |
*** |
bwolfe has quit IRC |
| [12:12:16] |
*** |
bwolfe has joined #openmrs |
| [12:12:16] |
*** |
ChanServ sets mode: +o bwolfe |
| [12:12:29] |
<OpenMRSBot> |
Recent updates in the world of openmrs: Upul Godage: OpenMRS database auto update <http://upulgsoc.wordpress.com/2008/08/13/openmrs-database-auto-update/> |
| [12:24:05] |
<sprasanna> |
bwolfe: where can i find any example for modifyiing global properties? |
| [12:24:41] |
<bwolfe> |
sprasanna: most global properties are edited on the global property administration screen |
| [12:25:07] |
<bwolfe> |
sprasanna: but ideally you just put the globalpropertyportlet on a separate jsp page |
| [12:30:13] |
<sprasanna> |
bwolfe: the default value for address format in global prop page is "general".. i can create something like this <bean id="addressHierarchyTemplate" class="org.openmrs.layout.web.address.AddressTemplate">... and i can set its properties in the module application context xml so what i want to now is when i load my module can i do something like this setProperty(layout.address.format) and setProperty("addresshierarchy") ?? |
| [12:35:18] |
<bwolfe> |
sprasanna: this might work: |
| [12:35:47] |
<bwolfe> |
create a bean in your modules application context like <bean id="addressTemplateUS" class="org.openmrs.layout.web.address.AddressTemplate"> (which is in openmrs-servlet.xml) |
| [12:36:09] |
<bwolfe> |
your class="..." would be your implementation/extend of AddressTemplate that overrides the getLines method |
| [12:36:42] |
<bwolfe> |
then the only thing you need to do is get your bean id into the addressSupport object |
| [12:38:31] |
<bwolfe> |
sprasanna: we probably need to change AddressSupport.setLayoutTemplates(List) to do: this.layoutTemplates.addAllThatArentInTheList(layoutTemplates); (instead of just doing this.layoutTemplates = layoutTemplates) |
| [12:40:15] |
<sprasanna> |
oh |
| [12:44:32] |
<bwolfe> |
sprasanna: does that make sens e? |
| [12:44:46] |
<bwolfe> |
sprasanna: did you follow the objects I was looking at in openmrs-servlet.xml ? |
| [12:44:51] |
<sprasanna> |
yeah it makes |
| [12:44:53] |
<sprasanna> |
sense |
| [12:45:27] |
<sprasanna> |
yeah now im little bit confused with the AddressSupport.setLayoutTemplates |
| [12:47:47] |
<bwolfe> |
sprasanna: how so ? |
| [12:48:03] |
<sprasanna> |
i mean i dont understand that part clearly |
| [12:48:16] |
<bwolfe> |
there will be only one AddressSupport object in the system because of how the contructor works |
| [12:48:36] |
<bwolfe> |
the AddressSupport.layoutTemplates are the objects available to the user |
| [12:49:09] |
<bwolfe> |
the user puts "usa" into their global properties table, and the "usa" template is looked up in the AddressSupport.layoutTemplates list |
| [12:51:17] |
<sprasanna> |
but modifying the module application context xml doesnt work or i dunno i have done any mistake... |
| [12:52:03] |
<bwolfe> |
sprasanna: what do you have in your modappcontext? is it committed to svn ? |
| [12:52:05] |
*** |
bmckown has quit IRC |
| [12:52:38] |
<sprasanna> |
i tried modyfing the property keys and values in addressTemplateGeneral |
| [12:52:52] |
<sprasanna> |
bwolfe: no its not committed yet i am just working on it |
| [12:53:32] |
<sprasanna> |
i did like this bwolfe: create a bean in your modules application context like <bean id="addressTemplateUS" class="org.openmrs.layout.web.address.AddressTemplate"> (which is in openmrs-servlet.xml) |
| [12:54:04] |
*** |
bmckown has joined #openmrs |
| [12:54:04] |
*** |
ChanServ sets mode: +o bmckown |
| [12:54:10] |
<sprasanna> |
i just modified the addressTemplateGeneral but i dont see any difference in my addresslayout page... |
| [12:54:20] |
<bwolfe> |
yours would be like: <bean id="addressTemplateWithHierarchy" class="org.openmrs.module.addresshierarchy.HierarchyTemplate"> |
| [12:55:13] |
<bwolfe> |
and you would need <bean parent="addressSupport" > <property name="layoutTemplate"><list><bean id="addressTemplateWithHierarchy"/></list></property></bean> |
| [12:55:18] |
<bwolfe> |
or something like htat |
| [12:56:00] |
<bwolfe> |
but with how hte core code is now, after installing your module, AddressSupport.layoutTemplates would /only/ contain your addressTemplateWithHierarchy object |
| [12:58:13] |
<sprasanna> |
is it?? you mean my xml will override the complete addressSupport bean? |
| [12:58:44] |
<bwolfe> |
upul: if liquibase won't run our sql file, perhaps we could just put a precondition on the liquibase xml doc http://liquibase.org/manual/databasechangelog to check the global_property database_version for the last one in the update-to-latest-mysql.diff. then we tell users to run the update-to-latest file and restart the war file |
| [12:59:01] |
<bwolfe> |
upul: then all future updates will just use the liquibase xml file |
| [12:59:22] |
<upul> |
bwolfe: that sounds good |
| [12:59:40] |
<bwolfe> |
sprasanna: <bean parent="addressSupport" > <property name="layoutTemplate"><list><bean id="addressTemplateWithHierarchy"/></list></property></bean> will use the already defined "addressSupport" object...and call setLayoutTemplates(list of one) |
| [13:02:02] |
<bwolfe> |
upul: or you create a customchange java class to run it. :-) http://liquibase.org/manual/custom_refactoring_class (assuming the user is running mysql of course) |
| [13:02:03] |
<OpenMRSBot> |
<http://ln-s.net/29Y5> (at liquibase.org) |
| [13:02:19] |
<sprasanna> |
bwolfe: please have a look at this http://pastebin.com/d70939caa |
| [13:02:59] |
<sprasanna> |
is there any mistake in tat xml? |
| [13:06:31] |
<bwolfe> |
sprasanna: yeah, thats what I was thinking |
| [13:06:51] |
<sprasanna> |
bwolfe: but unfortunately thats not working :( |
| [13:06:58] |
<bwolfe> |
sprasanna: expect you shouldn't have <bean id="addressTemplateWithHierarchy" class="org.openmrs.layout.web.address.AddressTemplate"> the class attribute should be your own class from your module that overrides the getLines method |
| [13:07:59] |
<bwolfe> |
brb |
| [13:08:10] |
*** |
bwolfe has quit IRC |
| [13:14:22] |
*** |
sioraiocht has joined #openmrs |
| [13:14:22] |
*** |
ChanServ sets mode: +v sioraiocht |
| [13:14:27] |
<OpenMRSBot> |
Recent updates in the world of openmrs: OpenMRS Tickets: Ticket #974 (enhancement created): OpenMRS database auto update using Liquibase <http://dev.openmrs.org/ticket/974> |
| [13:25:34] |
*** |
pearlbear has quit IRC |
| [13:25:39] |
*** |
nribeka has joined #openmrs |
| [13:34:47] |
*** |
upul has quit IRC |
| [13:43:28] |
*** |
pearlbear has joined #openmrs |
| [13:43:28] |
*** |
ChanServ sets mode: +v pearlbear |
| [14:06:48] |
*** |
bwolfe has joined #openmrs |
| [14:06:48] |
*** |
ChanServ sets mode: +o bwolfe |
| [14:14:36] |
*** |
bwolfe has quit IRC |
| [15:12:40] |
*** |
njero has joined #openmrs |
| [15:12:40] |
*** |
ChanServ sets mode: +v njero |
| [15:12:41] |
*** |
james_regen has quit IRC |
| [16:05:38] |
*** |
sioraiocht has quit IRC |
| [16:06:07] |
*** |
sioraiocht has joined #openmrs |
| [16:06:15] |
*** |
ChanServ sets mode: +v sioraiocht |
| [16:10:14] |
*** |
TorLye has quit IRC |
| [16:14:07] |
*** |
sgrannis has joined #openmrs |
| [16:14:15] |
*** |
ChanServ sets mode: +v sgrannis |
| [16:14:29] |
<sgrannis> |
nribeka: how's it going :) |
| [16:16:56] |
*** |
bmckown has quit IRC |
| [16:41:46] |
*** |
sgrannis has quit IRC |
| [16:42:37] |
*** |
pearlbear has quit IRC |
| [16:59:15] |
<nribeka> |
sgrannis i think you have some email that you need to answer in the dev list :D |
| [16:59:21] |
<nribeka> |
ups ... he's gone :D |
| [17:01:54] |
*** |
nribeka has quit IRC |
| [18:49:50] |
*** |
nribeka has joined #openmrs |
| [19:22:42] |
<OpenMRSBot> |
Recent updates in the world of openmrs: OpenMRS Tickets: Ticket #975 (task created): Trac doesn't keep log-in username/password <http://dev.openmrs.org/ticket/975> |
| [19:58:03] |
*** |
sprasanna has quit IRC |
| [21:19:39] |
*** |
upul has joined #openmrs |