Revision 41e2f693
Added by Cherian Mathew over 9 years ago
| eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/group/grantedauthority/CdmAuthorityCompositeViewer.java | ||
|---|---|---|
| 30 | 30 |
import eu.etaxonomy.cdm.persistence.hibernate.permission.CRUD; |
| 31 | 31 |
import eu.etaxonomy.cdm.persistence.hibernate.permission.CdmAuthority; |
| 32 | 32 |
import eu.etaxonomy.taxeditor.model.IDirtyMarkableSelectionProvider; |
| 33 |
import eu.etaxonomy.taxeditor.store.StoreUtil;
|
|
| 33 |
import eu.etaxonomy.taxeditor.model.MessagingUtils;
|
|
| 34 | 34 |
|
| 35 | 35 |
|
| 36 | 36 |
|
| ... | ... | |
| 146 | 146 |
// create a granted authrity from a cdm authority |
| 147 | 147 |
gai = cdma.asNewGrantedAuthority(); |
| 148 | 148 |
} catch (Exception e) {
|
| 149 |
StoreUtil.warningDialog("Parsing Error", this, "Could not parse authority string");
|
|
| 149 |
MessagingUtils.warningDialog("Parsing Error", this, "Could not parse authority string");
|
|
| 150 | 150 |
return; |
| 151 | 151 |
} |
| 152 | 152 |
//FIXME : this contains call will allow users to add authorities which differ only in CRUD operation choice. |
| 153 | 153 |
// need to have a comparator which only checks permission class and uuid |
| 154 | 154 |
if(cdmAuthorities.contains(gai) || newCdmAuthorities.contains(gai)) {
|
| 155 |
StoreUtil.warningDialog("Duplicate CDM Authority", this, "Chosen CDM Authority is already attached to current group");
|
|
| 155 |
MessagingUtils.warningDialog("Duplicate CDM Authority", this, "Chosen CDM Authority is already attached to current group");
|
|
| 156 | 156 |
} else {
|
| 157 | 157 |
group.addGrantedAuthority(gai); |
| 158 | 158 |
newCdmAuthorities.add(gai); |
Also available in: Unified diff
Moved all logging and dialog functionality to the new class MessagingUtils.
Refactoring code to adapt to above change.
Added new custom error dialog - CdmErrorDialog
Added runtime exception handling to the ApplicationWorkbenchAdvisor by adding a custom status handler