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:forEachitems="${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
|