Downloads Documentation Community Contribute Demo






Show Sidebar
Login | Register

IRC Logs

13. 08 2008

2008 8
Mo Tu We Th Fr Sa So
        1 2 3
4 5 6 7 8 9 10
11 12 13 14 15 16 17
18 19 20 21 22 23 24
25 26 27 28 29 30 31
[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