Merge lp:~echeverrifm/oemedical/oemedical_lab into lp:oemedical

Proposed by Federico Manuel Echeverri Choux
Status: Needs review
Proposed branch: lp:~echeverrifm/oemedical/oemedical_lab
Merge into: lp:oemedical
Diff against target: 330 lines (+18/-256)
3 files modified
oemedical_lab/__openerp__.py (+11/-54)
oemedical_lab/security/ir.model.access.csv (+1/-1)
oemedical_lab/views/oemedical_menu.xml (+6/-201)
To merge this branch: bzr merge lp:~echeverrifm/oemedical/oemedical_lab
Reviewer Review Type Date Requested Status
Alexandre Fayolle - camptocamp Needs Resubmitting
Pedro Manuel Baeza code review Approve
Yannick Vaucher @ Camptocamp english Needs Fixing
Review via email: mp+205519@code.launchpad.net

Description of the change

[IMP] Improve oemedical laboratory module (Fix some problems to install, modify menu sequence and remove comments of this module )

To post a comment you must log in.
Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :

Please set a description in __openerp__.py

l.71 does it still has errors ?

review: Needs Fixing (code review)
125. By Federico Manuel Echeverri Choux

[IMP] Add description of oemedical_lab module

126. By Federico Manuel Echeverri Choux

[IMP] oemedical_lab module [ remove some comment ]

Revision history for this message
Federico Manuel Echeverri Choux (echeverrifm) wrote :

> Please set a description in __openerp__.py
>
> l.71 does it still has errors ?

Hello, Sorry for the delay

I did this corrections

Regards

Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :

This sentence seams wrong.
l.64 Manager the request, creation and evaluation of laboratory analyses.

Wouldn't this be better? :

Manage request, creation and evaluation of laboratory analysis.

review: Needs Fixing (english)
127. By Federico Manuel Echeverri Choux

[IMP] Improve description of oemedical_lab module

Revision history for this message
Federico Manuel Echeverri Choux (echeverrifm) wrote :

>
> This sentence seams wrong.
> l.64 Manager the request, creation and evaluation of laboratory analyses.
>
> Wouldn't this be better? :
>
> Manage request, creation and evaluation of laboratory analysis.

Hello, Yannick

It's better and done

Regards

Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

LGTM.

Regards.

review: Approve (code review)
Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

Hello,

The management of the project has moved to Github: https://github.com/OCA/vertical-medical

Please migrate your merge proposal to Github. You may want to check https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub for an explanation on how to proceed.

Thanks for contributing to the project

review: Needs Resubmitting

Unmerged revisions

127. By Federico Manuel Echeverri Choux

[IMP] Improve description of oemedical_lab module

126. By Federico Manuel Echeverri Choux

[IMP] oemedical_lab module [ remove some comment ]

125. By Federico Manuel Echeverri Choux

[IMP] Add description of oemedical_lab module

124. By Federico Manuel Echeverri Choux

[IMP] Improve oemedical laboratory module (Fix some problems to install, modify menu sequence and remove comments of this module )

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'oemedical_lab/__openerp__.py'
2--- oemedical_lab/__openerp__.py 2014-01-15 10:20:02 +0000
3+++ oemedical_lab/__openerp__.py 2014-05-21 20:09:22 +0000
4@@ -34,78 +34,35 @@
5 ##############################################################################
6 {
7
8- 'name': 'OeMedical : Free Health and Hospital Information System',
9+ 'name': 'OeMedical : Laboratory Module',
10 'version': '1.0',
11 'author': "OeMEdical Team",
12 'category': 'Generic Modules/Others',
13- 'depends': ['oemedical_emr'],
14+ 'depends': ['oemedical'],
15 'application': True,
16 'description': """
17
18-About OeMedical
19----------------
20-
21-OeMedical is a multi-user, highly scalable, centralized Electronic
22-Medical Record (EMR) and Hospital Information System for openERP.
23-
24-OeMedical provides a free universal Health and Hospital Information System,
25-so doctors and institutions all over the world,
26-specially in developing countries will benefit from a centralized,
27-high quality, secure and scalable system.
28-
29-OeMedical at a glance:
30-
31- * Strong focus in family medicine and Primary Health Care
32-
33- * Major interest in Socio-economics (housing conditions, substance abuse,
34- education...)
35-
36- * Diseases and Medical procedures standards (like ICD-10 / ICD-10-PCS ...)
37-
38- * Patient Genetic and Hereditary risks : Over 4200 genes related to
39- diseases (NCBI / Genecards)
40-
41- * Epidemiological and other statistical reports
42-
43- * 100% paperless patient examination and history taking
44-
45- * Patient Administration
46- (creation, evaluations / consultations, history ... )
47-
48- * Doctor Administration
49-
50- * Lab Administration
51-
52- * Medicine / Drugs information (vademécum)
53-
54- * Medical stock and supply chain management
55-
56- * Hospital Financial Administration
57-
58- * Designed with industry standards in mind
59-
60- * Open Source : Licensed under AGPL
61+About OeMedical for laboratory
62+------------------------------
63+
64+Manage request, creation and evaluation of laboratory analysis.
65+
66
67 """,
68 "website": "http://launchpad.net/oemedical",
69 "licence": "AGPL v3",
70 "data": [
71- #'views/oemedical_lab_view.xml', # view has errors, please correct before enabling....
72+ 'views/oemedical_menu.xml',
73+ 'views/oemedical_lab_view.xml',
74 #'views/oemedical_lab_report.xml', # not checked yet....
75 'security/oemedical_security.xml',
76 'security/ir.model.access.csv',
77- 'oemedical_menu.xml',
78 ],
79 "demo": [
80
81 ],
82 'test':[
83- 'test/physician.yml',
84- 'test/patient.yml',
85- 'test/partners.yml',
86- 'test/insurance_plan.yml',
87- 'test/insurance.yml',
88- 'test/physician_speciality.yml'
89+
90 ],
91 'css': [
92
93@@ -117,6 +74,6 @@
94
95 ],
96 "active": False,
97- "installable": False,
98+ "installable": True,
99 }
100 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
101
102=== modified file 'oemedical_lab/security/ir.model.access.csv'
103--- oemedical_lab/security/ir.model.access.csv 2014-01-15 10:20:02 +0000
104+++ oemedical_lab/security/ir.model.access.csv 2014-05-21 20:09:22 +0000
105@@ -1,2 +1,2 @@
106 id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
107-access_oemedical_lab_group,model,model_oemedical_lab_group,group_oemedical_lab_user,1,1,1,0
108+access_oemedical_lab,model,model_oemedical_lab,group_oemedical_lab_user,1,1,1,0
109
110=== modified file 'oemedical_lab/views/oemedical_menu.xml'
111--- oemedical_lab/views/oemedical_menu.xml 2014-01-15 10:20:02 +0000
112+++ oemedical_lab/views/oemedical_menu.xml 2014-05-21 20:09:22 +0000
113@@ -1,211 +1,16 @@
114 <?xml version="1.0" encoding="UTF-8"?>
115 <openerp>
116 <data>
117- <menuitem id="oemedical_root" name="OeMedical" sequence="10" />
118-
119-
120-<!--Menu Patient-->
121-
122- <menuitem id="oemedical_patient_root" name="Patient" parent="oemedical_root"
123- sequence="10" />
124- <menuitem id="oemedical_patient" name=" Patient"
125- parent="oemedical_patient_root" sequence="10"
126- action="act_open_oemedical_patient_view" />
127-
128- <menuitem id="oemedical_family" name=" Family" parent="oemedical_patient_root"
129- sequence="20" action="act_open_oemedical_family_view" />
130-
131- <menuitem id="oemedical_family_member" name=" Family Member"
132- parent="oemedical_patient_root" sequence="30"
133- action="act_open_oemedical_family_member_view" />
134-
135-
136-<!--Appointment-->
137-
138- <menuitem id="oemedical_appoitment_root" name="Appoitment"
139- parent="oemedical_root" sequence="20" />
140-
141- <menuitem id="oemedical_appointment" name=" Appointment"
142- parent="oemedical_appoitment_root" sequence="10"
143- action="act_open_oemedical_appointment_view" />
144-
145-<!--Prescriptions-->
146-
147- <menuitem id="oemedical_prescription_root" name="Prescription"
148- parent="oemedical_root" sequence="30" />
149-
150- <menuitem id="oemedical_prescription_order" name="Prescription Order"
151- parent="oemedical_prescription_root" sequence="10"
152- action="act_open_oemedical_prescription_order_view" />
153
154 <!-- Laboratory -->
155 <menuitem name="Laboratory" id="oemedical_laboratory_menu"
156- parent="oemedical_root" sequence="4"/>
157-
158-
159-<!--Configurations-->
160-
161- <menuitem id="oemedical_root_sub" name="Configuration"
162- parent="oemedical_root" sequence="100" />
163-
164-<!-- Laboratory -->
165+ parent="oemedical.oemedical_root" sequence="90"/>
166+
167+
168+
169+<!-- Laboratory Config -->
170 <menuitem name="Laboratory" id="oemedical_conf_laboratory"
171- parent="oemedical_root_sub" sequence="3"/>
172-
173-
174-<!-- Diseases -->
175-
176- <menuitem id="oemedical_root_disease" name="Disease"
177- parent="oemedical_root_sub" sequence="10" />
178-
179-<!-- Menu of Pathology-->
180-
181- <menuitem id="oemedical_pathology_category" name=" Pathology Category"
182- parent="oemedical_root_disease" sequence="10"
183- action="act_open_oemedical_pathology_category_view" />
184-
185- <menuitem id="oemedical_pathology" name=" Pathology"
186- parent="oemedical_root_disease" sequence="30"
187- action="act_open_oemedical_pathology_view" />
188-
189- <menuitem id="oemedical_pathology_group" name=" Pathology Group"
190- parent="oemedical_root_disease" sequence="20"
191- action="act_open_oemedical_pathology_group_view" />
192-
193- <menuitem id="oemedical_root_disease_cat" name="Categories"
194- parent="oemedical_root_disease" sequence="100" />
195-
196- <menuitem id="oemedical_disease_group_members" name=" Disease Group"
197- parent="oemedical_root_disease_cat" sequence="10"
198- action="act_open_oemedical_disease_group_members_view" />
199-
200-<!-- Misc -->
201-
202- <menuitem id="oemedical_root_misc" name="Misc"
203- parent="oemedical_root_sub" sequence="200" />
204-
205-<!-- Occupation ( Config/misc/Occupation) )-->
206-
207- <menuitem id="oemedical_occupation" name=" Occupation"
208- parent="oemedical_root_misc" sequence="20"
209- action="act_open_oemedical_occupation_view" />
210-
211-<!-- Medical Specialty ( Config/misc/specialty) )-->
212-
213- <menuitem id="oemedical_specialty" name=" Medical Specialty"
214- parent="oemedical_root_misc" sequence="10"
215- action="act_open_oemedical_specialty_view" />
216-
217-<!-- Ethnicity ( Config/misc/Ethnicity) )-->
218-
219- <menuitem id="oemedical_ethnicity" name="Ethnicity" parent="oemedical_root_misc"
220- sequence="10" action="act_open_oemedical_ethnicity_view" />
221-
222-
223-
224-
225-
226-
227-<!--Medicaments-->
228-
229- <menuitem id="oemedical_medicament_root" name="Medicaments"
230- parent="oemedical_root_sub" sequence="50" />
231-
232-
233- <menuitem id="oemedical_medicament" name="Medicaments"
234- parent="oemedical_medicament_root" sequence="10"
235- action="act_open_oemedical_medicament_view" />
236-
237- <menuitem id="oemedical_medication_template" name=" Medication Template"
238- parent="oemedical_medicament_root" sequence="20"
239- action="act_open_oemedical_medication_template_view" />
240-
241- <menuitem id="oemedical_medication_frequencies" name="Medication Frequencies"
242- parent="oemedical_medicament_root" sequence="30"
243- action="act_open_oemedical_medication_dosage_view" />
244-
245- <menuitem id="oemedical_drug_route" name=" Drug Admin Route"
246- parent="oemedical_medicament_root" sequence="80"
247- action="act_open_oemedical_drug_route_view" />
248-
249- <menuitem id="oemedical_drug_form" name=" Drug Form"
250- parent="oemedical_medicament_root" sequence="90"
251- action="act_open_oemedical_drug_form_view" />
252-
253-
254- <menuitem id="oemedical_medicament_category" name=" Medicament Category"
255- parent="oemedical_medicament_root" sequence="100"
256- action="act_open_oemedical_medicament_category_view" />
257-
258-<!--Hospital Centers-->
259-
260- <menuitem id="oemedical_health_center_root" name="Health Center"
261- parent="oemedical_root_sub" sequence="10" />
262-
263- <menuitem id="oemedical_hospital_building" name=" Hospital Building"
264- parent="oemedical_health_center_root" sequence="10"
265- action="act_open_oemedical_hospital_building_view" />
266-
267- <menuitem id="oemedical_hospital_unit" name=" Hospital Unit"
268- parent="oemedical_health_center_root" sequence="20"
269- action="act_open_oemedical_hospital_unit_view" />
270-
271- <menuitem id="oemedical_hospital_ward" name=" Hospital Ward"
272- parent="oemedical_health_center_root" sequence="30"
273- action="act_open_oemedical_hospital_ward_view" />
274-
275- <menuitem id="oemedical_hospital_or" name=" Hospital Or"
276- parent="oemedical_health_center_root" sequence="40"
277- action="act_open_oemedical_hospital_or_view" />
278-
279- <menuitem id="oemedical_hospital_bed" name=" Hospital Bed"
280- parent="oemedical_health_center_root" sequence="50"
281- action="act_open_oemedical_hospital_bed_view" />
282-
283- <menuitem id="oemedical_operational_area" name=" Operational Area"
284- parent="oemedical_health_center_root" sequence="60"
285- action="act_open_oemedical_operational_area_view" />
286-
287- <menuitem id="oemedical_operational_sector" name=" Operational Sector"
288- parent="oemedical_health_center_root" sequence="70"
289- action="act_open_oemedical_operational_sector_view" />
290-
291-
292-<!-- Physician Conf -->
293-
294- <menuitem name="Health Professionals" id="oemedical_conf_physicians"
295- parent="oemedical_root_sub" sequence="20" />
296-
297- <menuitem id="oemedical_physician" name=" Physician" parent="oemedical_conf_physicians"
298- sequence="10" action="act_open_oemedical_physician_view" />
299-
300-
301-<!-- Insurance -->
302-
303- <menuitem name="Insurances" id="oemedical_conf_insurances"
304- parent="oemedical_root_misc" sequence="20" />
305-
306- <menuitem id="oemedical_insurance" name=" Insurance"
307- parent="oemedical_conf_insurances" sequence="10"
308- action="act_open_oemedical_insurance_view" />
309-
310- <menuitem id="oemedical_insurance_plan" name=" Insurance Plan"
311- parent="oemedical_conf_insurances" sequence="20"
312- action="act_open_oemedical_insurance_plan_view" />
313-
314-
315-
316-
317- <menuitem id="oemedical_diagnostic_hypothesis" name=" Diagnostic Hypothesis"
318- parent="oemedical_root_sub" sequence="10"
319- action="act_open_oemedical_diagnostic_hypothesis_view" />
320- <menuitem id="oemedical_predure" name=" Procedure" parent="oemedical_root_sub"
321- sequence="10" action="act_open_oemedical_procedure_view" />
322-
323- <menuitem id="oemedical_vaccination" name=" Vaccination"
324- parent="oemedical_root_sub" sequence="10"
325- action="act_open_oemedical_vaccination_view" />
326-
327+ parent="oemedical.oemedical_root_sub" sequence="150"/>
328
329 </data>
330 </openerp>