diff options
author | Noel Grandin <noel@peralex.com> | 2014-12-10 15:36:24 +0200 |
---|---|---|
committer | Noel Grandin <noelgrandin@gmail.com> | 2014-12-11 06:09:03 +0000 |
commit | 7557f23b31dcfb4d86c122bb34d9675c0db9a694 (patch) | |
tree | 5feb7be2b0841d8ee60d935cf2e29e9a01a31a27 /wizards/com/sun/star/wizards/form | |
parent | 808fd5fbd8868dfd95c8a38676815798fa2b79c4 (diff) |
java: reduce visibility of fields and methods
found by PMD
Change-Id: Id6737916b68ccbdbdeec5d314747a38410923ac6
Reviewed-on: https://gerrit.libreoffice.org/13409
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'wizards/com/sun/star/wizards/form')
7 files changed, 37 insertions, 37 deletions
diff --git a/wizards/com/sun/star/wizards/form/FieldLinker.java b/wizards/com/sun/star/wizards/form/FieldLinker.java index 4f7ffb101722..f5374c142db4 100644 --- a/wizards/com/sun/star/wizards/form/FieldLinker.java +++ b/wizards/com/sun/star/wizards/form/FieldLinker.java @@ -277,7 +277,7 @@ public class FieldLinker extends DBLimitedFieldSelection CurUnoDialog.setStepEnabled(IStep.intValue(), _bdoenable); } - class ItemListenerImpl implements com.sun.star.awt.XItemListener + private class ItemListenerImpl implements com.sun.star.awt.XItemListener { public void itemStateChanged(ItemEvent EventObject) diff --git a/wizards/com/sun/star/wizards/form/Finalizer.java b/wizards/com/sun/star/wizards/form/Finalizer.java index c1ed3a03db70..557814a431f6 100644 --- a/wizards/com/sun/star/wizards/form/Finalizer.java +++ b/wizards/com/sun/star/wizards/form/Finalizer.java @@ -33,11 +33,11 @@ import com.sun.star.wizards.ui.event.XTextListenerAdapter; public class Finalizer { - WizardDialog CurUnoDialog; - short curtabindex; - XRadioButton optModifyForm; - XTextComponent txtFormName; - FormDocument oFormDocument; + private WizardDialog CurUnoDialog; + private short curtabindex; + private XRadioButton optModifyForm; + private XTextComponent txtFormName; + private FormDocument oFormDocument; public Finalizer(WizardDialog _CurUnoDialog) { diff --git a/wizards/com/sun/star/wizards/form/FormConfiguration.java b/wizards/com/sun/star/wizards/form/FormConfiguration.java index ef46eeb1fc1f..e39f3cfd512f 100644 --- a/wizards/com/sun/star/wizards/form/FormConfiguration.java +++ b/wizards/com/sun/star/wizards/form/FormConfiguration.java @@ -40,17 +40,17 @@ import com.sun.star.wizards.ui.event.XItemListenerAdapter; public class FormConfiguration { - WizardDialog CurUnoDialog; - short curtabindex; - XRadioButton optOnExistingRelation; - XCheckBox chkcreateSubForm; - XRadioButton optSelectManually; - XFixedText lblRelations; - XListBox lstRelations; - String[] sreferencedTables; - CommandFieldSelection CurSubFormFieldSelection; - boolean bsupportsRelations; - RelationController oRelationController = null; + private WizardDialog CurUnoDialog; + private short curtabindex; + private XRadioButton optOnExistingRelation; + private XCheckBox chkcreateSubForm; + private XRadioButton optSelectManually; + private XFixedText lblRelations; + private XListBox lstRelations; + private String[] sreferencedTables; + private CommandFieldSelection CurSubFormFieldSelection; + private boolean bsupportsRelations; + private RelationController oRelationController = null; public FormConfiguration(WizardDialog _CurUnoDialog) { @@ -176,7 +176,7 @@ public class FormConfiguration Helper.setUnoPropertyValue(UnoDialog.getModel(optOnExistingRelation), PropertyNames.PROPERTY_ENABLED, Boolean.valueOf(bsupportsRelations && (chkcreateSubForm.getState() == 1))); } - public void toggleSteps() + private void toggleSteps() { if (chkcreateSubForm.getState() == 1) { diff --git a/wizards/com/sun/star/wizards/form/FormControlArranger.java b/wizards/com/sun/star/wizards/form/FormControlArranger.java index 41c975faceff..92e42997b02d 100644 --- a/wizards/com/sun/star/wizards/form/FormControlArranger.java +++ b/wizards/com/sun/star/wizards/form/FormControlArranger.java @@ -38,7 +38,7 @@ import com.sun.star.wizards.document.TimeStampControl; public class FormControlArranger { - public static final String LABELCONTROL = "LabelControl"; + private static final String LABELCONTROL = "LabelControl"; protected DatabaseControl[] DBControlList = null; private final XNameContainer xFormName; private final XMultiServiceFactory xMSF; diff --git a/wizards/com/sun/star/wizards/form/FormDocument.java b/wizards/com/sun/star/wizards/form/FormDocument.java index 8ef1936fb883..17698c29f40c 100644 --- a/wizards/com/sun/star/wizards/form/FormDocument.java +++ b/wizards/com/sun/star/wizards/form/FormDocument.java @@ -264,7 +264,7 @@ public class FormDocument extends TextDocument } } - public ControlForm getControlFormByName(String _sname) + private ControlForm getControlFormByName(String _sname) { for (int i = 0; i < oControlForms.size(); i++) { @@ -334,16 +334,16 @@ public class FormDocument extends TextDocument public class ControlForm { - XNameContainer xFormContainer; + private XNameContainer xFormContainer; GridControl oGridControl; - FormControlArranger oFormController; - int curArrangement; - FormDocument oFormDocument; - String Name; - Point aStartPoint; + private FormControlArranger oFormController; + private int curArrangement; + private FormDocument oFormDocument; + private String Name; + private Point aStartPoint; private Size aFormSize; - CommandMetaData oDBMetaData; - XPropertySet xPropertySet; + private CommandMetaData oDBMetaData; + private XPropertySet xPropertySet; public ControlForm(FormDocument _oFormDocument, String _sname, Point _astartPoint, Size _aFormSize) { @@ -432,12 +432,12 @@ public class FormDocument extends TextDocument } } - public Size getFormSize() + private Size getFormSize() { return aFormSize; } - public void setFormSize(Size _aSize) + private void setFormSize(Size _aSize) { aFormSize = _aSize; oFormController.setFormSize(aFormSize); @@ -537,7 +537,7 @@ public class FormDocument extends TextDocument } } - public void finalizeControls() + private void finalizeControls() { Control[] oLabelControls = getLabelControls(); Control[] oDBControls = getDatabaseControls(); diff --git a/wizards/com/sun/star/wizards/form/FormWizard.java b/wizards/com/sun/star/wizards/form/FormWizard.java index 5c57fe4cde6f..fe1791e6b3ae 100644 --- a/wizards/com/sun/star/wizards/form/FormWizard.java +++ b/wizards/com/sun/star/wizards/form/FormWizard.java @@ -47,7 +47,7 @@ public class FormWizard extends DatabaseObjectWizard private static String slblSelFields; private String sShowBinaryFields = PropertyNames.EMPTY_STRING; private String serrFormNameexists = PropertyNames.EMPTY_STRING; - public static final int SOMAIN_PAGE = 1; + private static final int SOMAIN_PAGE = 1; public static final int SOSUBFORM_PAGE = 2; public static final int SOSUBFORMFIELDS_PAGE = 3; public static final int SOFIELDLINKER_PAGE = 4; @@ -209,7 +209,7 @@ public class FormWizard extends DatabaseObjectWizard } } - public void buildSteps() + private void buildSteps() { curDBCommandFieldSelection = new CommandFieldSelection(this, curFormDocument.oMainFormDBMetaData, 92, slblFields, slblSelFields, slblTables, true, 34411); curDBCommandFieldSelection.addFieldSelectionListener(new FieldSelectionListener()); @@ -308,7 +308,7 @@ public class FormWizard extends DatabaseObjectWizard xDialog.endExecute(); } - public void insertFormRelatedSteps() + private void insertFormRelatedSteps() { addRoadmap(); int i = 0; @@ -370,10 +370,10 @@ public class FormWizard extends DatabaseObjectWizard serrFormNameexists = m_oResource.getResText(UIConsts.RID_FORM + 98); } - public class FieldSelectionListener implements com.sun.star.wizards.ui.XFieldSelectionListener + private class FieldSelectionListener implements com.sun.star.wizards.ui.XFieldSelectionListener { - protected int ID; + private int ID; // @Override public int getID() diff --git a/wizards/com/sun/star/wizards/form/StyleApplier.java b/wizards/com/sun/star/wizards/form/StyleApplier.java index cfb94438e70a..4a171e022fba 100644 --- a/wizards/com/sun/star/wizards/form/StyleApplier.java +++ b/wizards/com/sun/star/wizards/form/StyleApplier.java @@ -375,7 +375,7 @@ public class StyleApplier } } - public void applyDBControlProperties(int[] _iStyleColors) + private void applyDBControlProperties(int[] _iStyleColors) { try { |