profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/rubailly/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Bailly Rurangirwa rubailly Uganda

METS-Programme/openmrs-module-ugandaemr-reports 1

Reports for UgandaEMR distribution

rubailly/bahmni-config-c2c 0

Bahmni configuration and metadata for C2C

rubailly/bahmni-config-cambodia 0

Bahmni configuration and metadata for the OpenMRS Distribution for Cambodia

rubailly/bahmni-config-easydoc 0

Bahmni configuration for EasyDoc

rubailly/bahmni-config-haiti 0

Bahmni configuration for Haiti

rubailly/bahmni-config-hsc 0

Bahmni configuration and metadata for HSC

rubailly/bahmni-config-sanpedro 0

Bahmni configuration and metadata for the San Pedro Bahmni distribution

rubailly/bahmni-config-sgs 0

Bahmni config for Sustainable Global Surgery distribution

rubailly/bahmni-config-villaverde 0

Bahmni configuration and metadata for the Villa Verde Bahmni distribution

rubailly/bahmni-core 0

Core OpenMRS modules for Bahmni (including ERP & ELIS Atom Feed Clients)

Pull request review commentopenmrs/openmrs-sdk

SDK-288 SDK build-distro should build SPA application

 private String buildDistro(File targetDirectory, Artifact distroArtifact, Distro             try {                 ZipFile warfile = new ZipFile(new File(web, OPENMRS_WAR));                 File tempDir = new File(web, "WEB-INF");+                tempDir.mkdir();                 moduleInstaller.installModules(distroProperties.getModuleArtifacts(distroHelper, targetDirectory),                         new File(tempDir, WAR_FILE_MODULES_DIRECTORY_NAME).getAbsolutePath());                  File owasDir = new File(tempDir, "bundledOwas");-                owasDir.mkdirs();

How about web.mkdirs() above?

brandones

comment created time in 15 minutes

pull request commentopenmrs/openmrs-module-webservices.rest

RESTWS-821: Add REST resource with links to administration pages of installed modules

@dkayiwa Yes, I've tested it on my local OpenMRS server running version 2.4.0

BartlomiejRasztabiga

comment created time in 33 minutes

pull request commentopenmrs/openmrs-module-webservices.rest

RESTWS-821: Add REST resource with links to administration pages of installed modules

@BartlomiejRasztabiga did you test and confirm that it works well at runtime?

BartlomiejRasztabiga

comment created time in 37 minutes

pull request commentopenmrs/openmrs-module-webservices.rest

RESTWS-821: Add REST resource with links to administration pages of installed modules

@dkayiwa So I found a way to make it work. I've added legacyui-omod as a test dependency to all 2.0+ modules. I also had to bump it to version 1.8.3. Now it passes all the tests and can be deployed, the endpoint runs as well.

The only concern I have is why the PersonResource2_2Test failed on display property? It seems that adding legacyui-omod modified Person object for test purposes.

03f6a26

@dkayiwa What do you think about my recent solution?

BartlomiejRasztabiga

comment created time in an hour

pull request commentopenmrs/openmrs-module-webservices.rest

RESTWS-823: Add REST resource for Alert Management

@dkayiwa Have you got any change requests to this PR? :)

BartlomiejRasztabiga

comment created time in an hour

pull request commentopenmrs/openmrs-distro-referenceapplication

RATEST-91:Failing Test:Book Request Appointment Test https://issues.o…

Waiting approval to run actions cc: @k-joseph

insookwa

comment created time in an hour

push eventopenmrs/openmrs-module-reporting

dbaluku

commit sha ea7937e138ed899419c0fadd04a5546368e91e5c

REPORT-878:Enable RenderReport Method to accept multiple values for a parameter passed in a HTTP request (#216)

view details

push time in 2 hours

Pull request review commentopenmrs/openmrs-module-coreapps

'Patient Identifier column'

 function PatientSearchWidget(configuration){                             }                         });                     }+					/* Added Identifier type for the patient list in the main dashboard */+					else {+						const identifiersFiltered = patient.identifiers.filter(i => i.identifierType.uuid === "ffbd60fb-599a-4a57-b2f1-4b55847cd938" || i.identifierType.uuid === "3fa0990e-900d-4a80-a4c1-dcf01a0d4f9c");+						if (identifiersFiltered.length > 0) {

I spoke with Aldo about this, we agree on this approach.

MiguelAHPpih

comment created time in 2 hours

pull request commentopenmrs/openmrs-contrib-uitestframework

RATEST-92:RATEST-92:Failing Test: Contact Info Test

i shared the two fixes on single ticket

IMO this is not a better way to handle different issues into a single ticket and should not be encouraged. It will make tracking of work in the future very hard. Each issue should be handled in a separate ticket.

sherrif10

comment created time in 3 hours

pull request commentopenmrs/openmrs-contrib-uitestframework

RATEST-92:RATEST-92:Failing Test: Contact Info Test

True i agree but i shared the two fixes on single ticket ,so perhaps we can track them from there

sherrif10

comment created time in 3 hours

pull request commentopenmrs/openmrs-contrib-uitestframework

RATEST-92:RATEST-92:Failing Test: Contact Info Test

this is a bug within uitestframework and i needed to commit another pr in distro thank

Your right @sherrif10 ! It was my mistake on the comment I made.

I think such bugs(fixes)should be tracked in a separate ticket to for further reference.

sherrif10

comment created time in 3 hours

pull request commentopenmrs/openmrs-contrib-uitestframework

RATEST-92:RATEST-92:Failing Test: Contact Info Test

Where exactly @kdaud , this is a bug within uitestframework and i needed to commit another pr in distro thanks, Feel free to checkin here

sherrif10

comment created time in 3 hours

Pull request review commentopenmrs/openmrs-sdk

SDK-288 SDK build-distro should build SPA application

 private String buildDistro(File targetDirectory, Artifact distroArtifact, Distro             try {                 ZipFile warfile = new ZipFile(new File(web, OPENMRS_WAR));                 File tempDir = new File(web, "WEB-INF");+                tempDir.mkdir();                 moduleInstaller.installModules(distroProperties.getModuleArtifacts(distroHelper, targetDirectory),                         new File(tempDir, WAR_FILE_MODULES_DIRECTORY_NAME).getAbsolutePath());                  File owasDir = new File(tempDir, "bundledOwas");-                owasDir.mkdirs();

Yeah, when I did the refactor of the setup task I eventually realized that a owasDir.mkdirs() call was actually accidentally being used to create the server directory itself. This made it very hard to reason about what was happening when. That gave me the belief that mkdirs should be used sparingly. It opens the door for code to get messy and dependent on what should be side-effects.

brandones

comment created time in 3 hours

pull request commentopenmrs/openmrs-distro-referenceapplication

RATEST-92:Failing Test: Contact Info Test

Untill this bug is fixed with in https://github.com/openmrs/openmrs-contrib-uitestframework/pull/45#issuecomment-867672505 , then this ticket will be able to work as expected, so i fixed two at once cc @k-joseph

sherrif10

comment created time in 3 hours

PullRequestEvent

Pull request review commentopenmrs/openmrs-module-reporting

REPORT-873: SQL reports to use cursors to iterate through datasets.

 void writeDataSet(DataSet dataset, OutputStream out, String textDelimiter, Strin 		w.write(lineEnding);
 
 		// data rows
-		for (DataSetRow row : dataset) {
-			for (Iterator<DataSetColumn> i = columns.iterator(); i.hasNext();) {
-				DataSetColumn column = i.next();
-				Object colValue = row.getColumnValue(column);
-				w.write(textDelimiter);
-				if (colValue != null) {
-                    String toPrint;
-					if (colValue instanceof Cohort) {
-                        toPrint = escape(Integer.toString(((Cohort) colValue).size()));
-					}
-					else if (colValue instanceof IndicatorResult) {
-                        toPrint = ((IndicatorResult) colValue).getValue().toString();
-					}
-					else if (dateFormat != null && colValue instanceof Date) {
-                        toPrint = dateFormat.format((Date)colValue);
-                    }
-					else {
-						// this check is because a logic EmptyResult .toString() -> null
-						String temp = escape(colValue.toString());
-						if (temp != null) {
-                            toPrint = temp;
-                        } else {
-                            toPrint = "";
-                        }
-					}
-                    w.write(filterBlacklist(toPrint, blacklist));
-				}
-				w.write(textDelimiter);
-				if (i.hasNext()) {
-					w.write(fieldDelimiter);
-				}
-			}
-			w.write(lineEnding);
-		}
-
+        if (dataset instanceof IterableSqlDataSet) {

Sure - I don't want to get too bogged down in this - I was just trying to make suggestions, not trying to be a blocker. If the entire PR is ready for review again, let me know.

icrc-loliveira

comment created time in 5 hours

pull request commentopenmrs/openmrs-contrib-uitestframework

RATEST-92:RATEST-92:Failing Test: Contact Info Test

Was it an oversight to leave the @ignore annotation on the test method that the ticket is resolving ?

sherrif10

comment created time in 5 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-117:EditPatientRelationshipPage

+package org.openmrs.reference.page;++import org.openmrs.uitestframework.page.Page;+import org.openqa.selenium.By;+import org.openqa.selenium.support.ui.Select;++public class EditPatientRelationshipPage  extends Page {++    private  HomePage homePage;+    private static String name = "john";+    private  static  final By SELECT_RELATIONSHIP_TYPE = By.id("relationship_type");+    private static final By NAME = By.cssSelector("button, input");+    private static final By NEXT_BUTTON = By.id("next-button");+    private static final By CONFIRM_BUTTON = By.id("registration-submit");+    ++    public EditPatientRelationshipPage(Page parent) {+        super(parent);+    }++    public void clickOnSelectRelationshipType() throws InterruptedException{+        clickOn(SELECT_RELATIONSHIP_TYPE);+        Select relationshipType = new Select(driver.findElement(By.id("relationship_type")));+        relationshipType.selectByVisibleText("Doctor");+        setTextToFieldNoEnter(NAME,name);+        clickOn(NEXT_BUTTON);

setTextToFieldNoEnter(NAME,name); allows the input of the string "john"

sherrif10

comment created time in 6 hours

pull request commentopenmrs/openmrs-distro-referenceapplication

RATEST-91:Failing Test:Book Request Appointment Test https://issues.o…

@insookwa have you tested locally on all instances and the test is passing ?

insookwa

comment created time in 6 hours

Pull request review commentopenmrs/openmrs-module-coreapps

RA-1926: Merge Visits: Remove visitId from URL to prevent NPE if that visit was merged

 public Object controller(@RequestParam("patientId") Patient patient,      public String post(@RequestParam("patientId") Patient patient,                        @RequestParam("mergeVisits") List<Integer> mergeVisits,-					   @RequestParam(value = "returnUrl", required = false) String returnUrl,+                       @RequestParam(value = "returnUrl", required = false) String returnUrl,                        @SpringBean AdtService service,                        UiUtils ui,-                       HttpServletRequest request, PageModel model){+                       HttpServletRequest request, PageModel model) { +        List<String> mergedVisitIds = new ArrayList<String>();         if (patient.isVoided() || patient.isPersonVoided()) {             return "redirect:" + ui.pageLink("coreapps", "patientdashboard/deletedPatient",-					SimpleObject.create("patientId", patient.getId().toString() ));+                    SimpleObject.create("patientId", patient.getId().toString()));         } -        if (mergeVisits!=null && mergeVisits.size() > 0 ){+        if (mergeVisits != null && mergeVisits.size() > 0) {             Visit mergedVisit = service.mergeConsecutiveVisits(mergeVisits, patient);-            if (mergedVisit != null){+            if (mergedVisit != null) {                 request.getSession().setAttribute(AppUiConstants.SESSION_ATTRIBUTE_INFO_MESSAGE, ui.message("coreapps.task.mergeVisits.success"));                 request.getSession().setAttribute(AppUiConstants.SESSION_ATTRIBUTE_TOAST_MESSAGE, "true");-            }else{+            } else {                 request.getSession().setAttribute("emr.errorMessage", ui.message("coreapps.task.mergeVisits.error"));                 request.getSession().setAttribute(AppUiConstants.SESSION_ATTRIBUTE_TOAST_MESSAGE, "true");             }++            for (int i = 1; i < mergeVisits.size(); i++) {

Did you intentionally start with index 1 instead of 0?

icrc-jfigueiredo

comment created time in 7 hours

Pull request review commentopenmrs/openmrs-sdk

SDK-288 SDK build-distro should build SPA application

 private String buildDistro(File targetDirectory, Artifact distroArtifact, Distro             try {                 ZipFile warfile = new ZipFile(new File(web, OPENMRS_WAR));                 File tempDir = new File(web, "WEB-INF");+                tempDir.mkdir();                 moduleInstaller.installModules(distroProperties.getModuleArtifacts(distroHelper, targetDirectory),                         new File(tempDir, WAR_FILE_MODULES_DIRECTORY_NAME).getAbsolutePath());                  File owasDir = new File(tempDir, "bundledOwas");-                owasDir.mkdirs();

Was the change from mkdirs to mkdir intentional?

brandones

comment created time in 7 hours

PR opened openmrs/openmrs-distro-referenceapplication

RATEST-91:Failing Test:Book Request Appointment Test https://issues.o…

I deleted the other pull request and all the changes are covered already. I have removed the @ignore
kindly review https://issues.openmrs.org/browse/RATEST-91 @k-joseph @kdaud

+0 -2

0 comment

1 changed file

pr created time in 7 hours

PR closed openmrs/openmrs-distro-referenceapplication

Ratest 91:RATEST-91:Failing Test: Book Request Appointment Test

Fixed this test. please kindly review https://issues.openmrs.org/browse/RATEST-91

+0 -0

2 comments

0 changed file

insookwa

pr closed time in 8 hours

PR closed openmrs/openmrs-distro-referenceapplication

RATEST-92:Failing Test: Contact Info Test

https://issues.openmrs.org/browse/RATEST-92 Draft for checking the validationError

+8 -7

1 comment

2 changed files

sherrif10

pr closed time in 10 hours

pull request commentopenmrs/openmrs-distro-referenceapplication

RATEST-92:Failing Test: Contact Info Test

The Fix was done in https://github.com/openmrs/openmrs-contrib-uitestframework/pull/45 , since the element was not the collect one, so am closing this

sherrif10

comment created time in 10 hours