2013 | Niklaus.20131217-fix-sequences-00 >>
---
@session.persistent_user_input(:drugs)
by using the url, e.g. ?ean=ean_01,ean_02,ean_03
Continue work. Example is still interaction is between Marcoumar and Aspirin. Open problems are:
Example at matrix.epha.org is:
Example from Zeno
Decided to make unit-tests for the working example teraction_chooser and then trying to add different entites or override compose_list/header.
Okay. Now I can reproduce my error inside a test-case. See Attach:test_interaction_list.txt
Error: test_interaction_list_to_html(TestComposite) NoMethodError: undefined method `set_attributes' for nil:NilClass /opt/src/htmlgrid/lib/htmlgrid/composite.rb:260:in `to_html' test/test_interaction_list.rb:232:in `test_interaction_list_to_html' 229: '<TABLE cellspacing="0">'+ 230: '<TR><TD class="small">fachinfo</TD><TD class="interaction-atc">atc</TD><TD class="small">delete</TD></TR></TABLE>'+ 231: '</TD></TR><TR><TD>Asp</TD></TR></TABLE>' => 232: assert_equal(expected, composite.to_html(CGI.new)) 233: puts "\n" + composite.to_html(CGI.new) 234: end 235: end
Added new tests to htmlgrid as constructing a list like needed in interactions is not trivial. See commit Added more unit-test for working with list
Refactoring displays correct info now on $stdout, but nothing is displayed. There must be a missing element. Fixed this problem by appending @value << InteractionChooserDrugList.new
. Next problem ist that Alle löschen
appears twice in the layout. Fixed this problem. Now deleting a medicament works now too. Have seen that sometime when the list is empty, you cannot add the old medicament. Adding another worked.
Must filter the interactions, as they are show too many times. Now the whole stuff looks quite good. I still prefer that we should only display the interactions that stem from adding a new drug, as I think this corresponds better to the doctors thinking which will try to add drugs in decreasing order of importance and will probably replace the last one added by one which has less interactions. See
I still have a few too many lines at the bottome. Clean up. Also used the background color for the header. Pushed commits.
But I think the interactions should have some space on top and the left to clearly distinguis them. See
Removed switch to instant search via commit Clean up by removing old search files
Then I tried to remove on a separate branch all references to the old search via the commit Remove not needed files for old search form. Then I tested search by trademark "Xeljanz" and could not find anything. There I did not merge this branch into master.