/ - Diff - EDIT - EDIT Project Management

« Previous | Next »

Revision 0ce0edd2

ref #7349 prevent from setting nbew persons as value to person select - fixes problem with person fields being mixed up

                 personSelect.clear();
             });
             selectOrNewContainer.addComponents(personSelect, personSelectConfirmButton, newPersonButton);
             newPersonButton.addClickListener(e -> {
                 setValue(Person.NewInstance());
             });
             newPersonButton.addClickListener(e -> createNewPerson());
             // edit person
             addStyledComponent(titleCacheField);
-...
             return Person.class;
+        }
         private void createNewPerson(){
             Person p = Person.NewInstance();
             setValue(p);
+        }
         @Override
         public void setValue(Person person){
             super.setValue(person);
     //        PersonFieldAccess personWrapper = new PersonFieldAccess(person);
     //        super.setValue(personWrapper);
             personSelect.setValue(person);
             if(person != null && person.getId() != 0){
                 personSelect.setValue(person);
+            }
+        }
         /**

Also available in: Unified diff