Changeset 28444

Show
Ignore:
Timestamp:
01/02/08 19:53:47 (1 year ago)
Author:
sfermigier
Message:

Forward port.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-archive-facade/src/main/java/org/nuxeo/ecm/platform/archive/service/NXArchiveFactoryService.java

    r20218 r28444  
    7070    } 
    7171 
    72     @SuppressWarnings("static-access") 
    7372    public Class<ArchiveRecordFactory> getArchiveRecordFactoryKlass() { 
    74         return this.archiveRecordFactoryKlass; 
     73        return archiveRecordFactoryKlass; 
    7574    } 
    7675 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-audit-facade/src/main/java/org/nuxeo/ecm/platform/audit/ejb/NXAuditMessageListener.java

    r25118 r28444  
    127127        NXAuditEvents service = NXAudit.getNXAuditEventsService(); 
    128128        Set<String> eventIds = service.getAuditableEventNames(); 
    129         if (eventIds.contains(eventId)) { 
    130             return true; 
    131         } 
    132         return false; 
     129        return eventIds.contains(eventId); 
    133130    } 
    134131 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-content-template-manager/src/main/java/org/nuxeo/ecm/platform/content/template/factories/BaseContentFactory.java

    r23426 r28444  
    1616    protected CoreSession session; 
    1717 
    18     protected Boolean initSession(DocumentModel eventDoc) 
    19     { 
     18    protected Boolean initSession(DocumentModel eventDoc) { 
    2019        String sessionId = eventDoc.getSessionId(); 
    2120        session = CoreInstance.getInstance().getSession(sessionId); 
    22         if (session==null) 
    23             return false; 
    24         return true; 
     21        return session != null; 
    2522    } 
    2623 
    27     public void createContentStructure(DocumentModel eventDoc) throws ClientException { 
     24    public void createContentStructure(DocumentModel eventDoc) 
     25            throws ClientException { 
    2826        throw new NotImplementedException(); 
    2927    } 
    3028 
    31     public Boolean initFactory(Map<String, String> options, List<TemplateItemDescriptor> template) { 
     29    public Boolean initFactory(Map<String, String> options, 
     30            List<TemplateItemDescriptor> template) { 
    3231        throw new NotImplementedException(); 
    3332    } 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-directory-api/src/main/java/org/nuxeo/ecm/directory/impl/DirectoryClientImpl.java

    r25638 r28444  
    3939public class DirectoryClientImpl implements DirectoryClient { 
    4040 
    41     /** 
    42      * 
    43      */ 
    4441    private static final long serialVersionUID = -1170479958816244690L; 
    4542 
    46     private long sessionId; 
     43    private final long sessionId; 
    4744 
    4845    private transient DirectoryManager manager; 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-directory-multi/src/test/java/org/nuxeo/ecm/directory/multi/TestMemoryDirectory.java

    r26508 r28444  
    298298    } 
    299299 
    300     protected List<String> entryIds(List<DocumentModel> entries) { 
     300    protected static List<String> entryIds(List<DocumentModel> entries) { 
    301301        List<String> ids = new ArrayList<String>(entries.size()); 
    302302        for (DocumentModel entry : entries) { 
    …… 
    315315        dir.createEntry(e2); 
    316316 
    317         Map<String, Object> filter = Collections.<String, Object>emptyMap(); 
    318         Set<String> fulltext = Collections.<String>emptySet(); 
     317        Map<String, Object> filter = Collections.emptyMap(); 
     318        Set<String> fulltext = Collections.emptySet(); 
    319319        Map<String, String> orderBy = new LinkedHashMap<String, String>(); 
    320320        DocumentModelList entries; 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-domsync-core/src/main/java/org/nuxeo/ecm/platform/domsync/core/DOMSynchronizer.java

    r28225 r28444  
    6161    private DOMMutationEvent currentEvent; /* for debug/test purposes */ 
    6262    private List<IDOMMutationListener> listeners = new ArrayList<IDOMMutationListener>(); 
    63      
     63 
    6464    public DOMSynchronizer(Document document, IDOMSupport domSupport) { 
    6565        this.document = document; 
    6666        this.domSupport = domSupport; 
    6767    } 
    68      
     68 
    6969    /** 
    7070     * Document event handler 
    …… 
    7676        MutationEvent event = (MutationEvent) evt; 
    7777        String type = event.getType(); 
    78          
     78 
    7979        if (DOM_CHARACTER_DATA_MODIFIED.equals(type)) { 
    8080            Node target = (Node) event.getTarget(); 
    8181            String newValue = event.getNewValue(); 
    82             dispatchEvent(new DOMCharacterDataModifiedEvent(DOMUtil.computeNodeXPath(document, target), newValue)); 
    83              
     82            dispatchEvent(new DOMCharacterDataModifiedEvent( 
     83                    DOMUtil.computeNodeXPath(document, target), newValue)); 
     84 
    8485        } else if (DOM_NODE_INSERTED.equals(type)) { 
    8586            Node target = event.getRelatedNode(); 
    8687            Node insertedNode = (Node)event.getTarget(); 
    8788            int position = DOMUtil.getNodePosition(insertedNode); 
    88             ArrayList<DOMNodeInsertedEvent> list = new ArrayList<DOMNodeInsertedEvent>(); 
    89             buildFragmentInsertedEvents(DOMUtil.computeNodeXPath(document, target), insertedNode, position, list); 
    90             for(Iterator<DOMNodeInsertedEvent> i = list.iterator(); i.hasNext(); ) { 
    91                 dispatchEvent(i.next()); 
    92             } 
    93              
     89            List<DOMNodeInsertedEvent> list = new ArrayList<DOMNodeInsertedEvent>(); 
     90            buildFragmentInsertedEvents(DOMUtil.computeNodeXPath(document, target), 
     91                    insertedNode, position, list); 
     92            for (DOMNodeInsertedEvent aList : list) { 
     93                dispatchEvent(aList); 
     94            } 
     95 
    9496        } else if (DOM_NODE_REMOVED.equals(type)) { 
    9597            Node target = (Node) event.getTarget(); 
    96             dispatchEvent(new DOMNodeRemovedEvent(DOMUtil.computeNodeXPath(document, target))); 
    97              
     98            dispatchEvent(new DOMNodeRemovedEvent( 
     99                    DOMUtil.computeNodeXPath(document, target))); 
     100 
    98101        } else if (DOM_ATTR_MODIFIED.equals(type)) { 
    99102            Node target = (Node) event.getTarget(); 
    100             dispatchEvent(new DOMAttrModifiedEvent(DOMUtil.computeNodeXPath(document, target), event.getAttrName(), event.getAttrChange(), event.getNewValue())); 
    101              
     103            dispatchEvent(new DOMAttrModifiedEvent( 
     104                    DOMUtil.computeNodeXPath(document, target), event.getAttrName(), event.getAttrChange(), event.getNewValue())); 
     105 
    102106        } else { 
    103107            System.err.println("!Unsupported event type "+type); 
    104108        } 
    105109    } 
    106      
     110 
    107111    private void buildFragmentInsertedEvents(String baseXPath, Node node, int position, List<DOMNodeInsertedEvent> list) { 
    108112        if (node instanceof Text) { 
    109             list.add(new DOMNodeInsertedEvent(baseXPath, "#text"+((Text)node).getData(), position)); 
     113            list.add(new DOMNodeInsertedEvent(baseXPath, 
     114                    "#text" + ((Text) node).getData(), position)); 
    110115        } else if (node instanceof Element) { 
    111             list.add(new DOMNodeInsertedEvent(baseXPath, DOMUtil.getElementOuterNoChildren((Element) node), position)); 
    112             if(node.hasChildNodes()) { 
    113                 baseXPath = baseXPath + DOMUtil.computeNodeXPath(node.getParentNode(),node); 
     116            list.add(new DOMNodeInsertedEvent(baseXPath, 
     117                    DOMUtil.getElementOuterNoChildren((Element) node), position)); 
     118            if (node.hasChildNodes()) { 
     119                baseXPath += DOMUtil.computeNodeXPath(node.getParentNode(), node); 
    114120                node = node.getFirstChild(); 
    115121                position = 0; 
    116                 while(node != null) { 
    117                     buildFragmentInsertedEvents(baseXPath, node, position, list); 
     122                while (node != null) { 
     123                    buildFragmentInsertedEvents(baseXPath, node, position, 
     124                            list); 
    118125                    node = node.getNextSibling(); 
    119126                    ++position; 
    …… 
    125132        } 
    126133    } 
    127              
     134 
    128135    private void dispatchEvent(DOMMutationEvent event) { 
    129136        if(dispatchLevel != 0) { 
    …… 
    136143        } 
    137144        IDOMMutationListener[] list = listeners.toArray(new IDOMMutationListener[listeners.size()]); 
    138         for(int i = 0; i < list.length; ++i) { 
    139             list[i].handleEvent(event); 
    140         } 
    141     } 
    142      
     145        for (IDOMMutationListener listener : list) { 
     146            listener.handleEvent(event); 
     147        } 
     148    } 
     149 
    143150    public void addMutationListener(IDOMMutationListener listener) { 
    144151        if (!listeners.contains(listener)) { 
    …… 
    158165        ++dispatchLevel; 
    159166        try { 
    160             Node target = DOMUtil.findNodeByXPath(document, event.getTarget());         
     167            Node target = DOMUtil.findNodeByXPath(document, event.getTarget()); 
    161168            if (target == null) { 
    162169                System.err.println("!Null target for "+event.getTarget()); 
    163170                return; 
    164171            } 
    165              
     172 
    166173            if (event instanceof DOMNodeInsertedEvent) { 
    167174                if(!(target instanceof Element) && !(target instanceof Document) ) { 
    …… 
    183190                    target.appendChild(docFragment); 
    184191                } 
    185                  
     192 
    186193            } else if (event instanceof DOMNodeRemovedEvent) { 
    187194                if(!(target instanceof Element) && !(target instanceof CharacterData) && !(target instanceof ProcessingInstruction)) { 
    188195                    System.err.println("!Unsupported target node type"); 
    189                     return;                 
     196                    return; 
    190197                } 
    191198                target.getParentNode().removeChild(target); 
    192                  
     199 
    193200            } else if (event instanceof DOMAttrModifiedEvent) { 
    194201                if(!(target instanceof Element)) { 
    195202                    System.err.println("!Unsupported target node type"); 
    196                     return;                 
     203                    return; 
    197204                } 
    198205                String attrName = ((DOMAttrModifiedEvent)event).getAttrName(); 
    …… 
    212219                    } 
    213220                } 
    214                  
     221 
    215222            } else if (event instanceof DOMCharacterDataModifiedEvent) { 
    216223                String data = ((DOMCharacterDataModifiedEvent)event).getNewValue(); 
    …… 
    218225                    ((CharacterData)target).setData(data); 
    219226                } else if (target instanceof ProcessingInstruction) { 
    220                     ((ProcessingInstruction)target).setData(data);                 
    221                 } else { 
    222                     System.err.println("!Unsupported target node type"); 
    223                 } 
    224                  
     227                    ((ProcessingInstruction)target).setData(data); 
     228                } else { 
     229                    System.err.println("!Unsupported target node type"); 
     230                } 
     231 
    225232            } else { 
    226233                System.err.println("!Unsupported event "+event); 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-events-api/src/main/java/org/nuxeo/ecm/platform/events/api/delegate/DocumentMessageProducerBusinessDelegate.java

    r22664 r28444  
    2828/** 
    2929 * Document message producer service delegate. 
    30  *  
     30 * 
    3131 * <p> 
    3232 * Utility class giving access to the <code>DocumentMessageProducer</code> 
    3333 * service. 
    3434 * </p> 
    35  *  
     35 * 
    3636 * @author <a href="mailto:ja@nuxeo.com">Julien Anguenot</a> 
    37  *  
     37 * 
    3838 */ 
    3939public final class DocumentMessageProducerBusinessDelegate implements 
    …… 
    4747    /** 
    4848     * Returns the remote document message producer service. 
    49      *  
     49     * 
    5050     * @return a <code>DocumentMessageProducer</code> service instance. 
    5151     * @throws DocumentMessageProducerException if an error occured at lookup 
    5252     *             time. 
    5353     */ 
    54     public static final DocumentMessageProducer getRemoteDocumentMessageProducer() 
     54    public static DocumentMessageProducer getRemoteDocumentMessageProducer() 
    5555            throws DocumentMessageProducerException { 
    5656        DocumentMessageProducer producer = null; 
    …… 
    6565    /** 
    6666     * Returns a local document message producer service. 
    67      *  
     67     * 
    6868     * @return a <code>DocumentMessageProducer</code> service instance. 
    6969     */ 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-notification-facade/src/main/java/org/nuxeo/ecm/platform/ec/notification/ejb/NotificationMessageListener.java

    r26373 r28444  
    333333 
    334334        } 
    335  
    336335    } 
    337336 
    …… 
    340339        deletionEvents.add("aboutToRemove"); 
    341340        deletionEvents.add("documentRemoved"); 
    342         if (deletionEvents.contains(message.getEventId())){ 
    343             return true; 
    344         } 
    345         return false; 
     341        return deletionEvents.contains(message.getEventId()); 
    346342    } 
    347343 
    …… 
    408404            throw new ClientException("Failed to send notification email ", e); 
    409405        } 
    410  
    411406    } 
    412407 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-relations-jena-plugin/src/main/java/org/nuxeo/ecm/platform/relations/jena/JenaGraph.java

    r28267 r28444  
    274274                        connection.getDriver().setDoCompressURI(true); 
    275275                    } 
    276                     if (databaseTransactionEnabled || false) { 
     276                    if (databaseTransactionEnabled) { 
    277277                        connection.getDriver().setIsTransactionDb(true); 
    278278                    } 
    …… 
    282282                graph = m.createModel(name); 
    283283            } 
    284             graph.setNsPrefixes(this.namespaces); 
     284            graph.setNsPrefixes(namespaces); 
    285285            // use baseConnection so that it is closed instead of the jena one 
    286286            // (to let the pool handled closure). 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-search-api/src/main/java/org/nuxeo/ecm/core/search/api/client/indexing/nxcore/IndexingHelper.java

    r26092 r28444  
    4747/** 
    4848 * This class provides helpers for (re)(un)indexing processes 
    49  *  
     49 * 
    5050 * @author <a href="mailto:gracinet@nuxeo.com">Georges Racinet</a> 
    51  *  
     51 * 
    5252 */ 
    5353public final class IndexingHelper { 
    …… 
    7373    /** 
    7474     * Delete one document from the document indexes. 
    75      *  
     75     * 
    7676     * @param docModel : the actual document model to unindex. 
    7777     * @param service : the search service. 
    …… 
    146146                systemSession.login(); 
    147147                return systemSession; 
    148             } else { 
    149                 log.debug("Bound to an IndexingThead. No need reauthenticate...."); 
    150             } 
     148            } 
     149            log.debug("Bound to an IndexingThead. No need reauthenticate...."); 
    151150            return null; 
    152151        } catch (Exception e) { 
    …… 
    167166     * index a newly created document as part as the same transaction that 
    168167     * created the document. 
    169      *  
     168     * 
    170169     * @param docModel 
    171170     * @param managedSessionId 
    …… 
    187186     * Start a recursive indexing process using a new CoreSession instance that 
    188187     * will automatically be closed at the end og the indexing process. 
    189      *  
     188     * 
    190189     * @param docModel 
    191190     * @throws Exception 
    …… 
    288287 
    289288        while (true) { 
    290  
    291289            for (ResultItem item : rset) { 
    292290                String key = (String) item.get(BuiltinDocumentFields.FIELD_DOC_UUID); 
    …… 
    300298            } 
    301299 
    302             if (rset.hasNextPage()) 
     300            if (rset.hasNextPage()) { 
    303301                try { 
    304302                    rset = rset.nextPage(); 
    …… 
    306304                    throw new IndexingException(se); 
    307305                } 
    308             else { 
     306            } else { 
    309307                break; 
    310308            } 
    311  
    312         } 
    313  
     309        } 
    314310    } 
    315311 
    …… 
    334330 
    335331    private static boolean isBoundToIndexingThread() { 
    336         if (Thread.currentThread() instanceof IndexingThread) { 
    337             return true; 
    338         } 
    339         return false; 
     332        return Thread.currentThread() instanceof IndexingThread; 
    340333    } 
    341334 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-search-api/src/main/java/org/nuxeo/ecm/core/search/api/client/indexing/resources/AbstractIndexableResource.java

    r22566 r28444  
    4040    private static final Log log = LogFactory.getLog(AbstractIndexableResource.class); 
    4141 
     42    private static final long serialVersionUID = 7457965764411626518L; 
     43 
    4244    protected String name; 
    4345 
    …… 
    6769    protected boolean isBoundToIndexingThread() { 
    6870        if (boundToIndexingThread == null) { 
    69             if (Thread.currentThread() instanceof IndexingThread) { 
    70                 boundToIndexingThread = true; 
    71             } else { 
    72                 boundToIndexingThread = false; 
    73             } 
     71            boundToIndexingThread = Thread.currentThread() instanceof IndexingThread; 
    7472        } 
    7573        return boundToIndexingThread; 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-search-compass-plugin/src/main/java/org/nuxeo/ecm/core/search/backend/compass/join/QuerySplitter.java

    r22551 r28444  
    9393    } 
    9494 
    95     public boolean isJoin(Predicate p) { 
    96         return (p.lvalue instanceof Reference) 
    97             && (p.rvalue instanceof Reference); 
     95    public static boolean isJoin(Predicate p) { 
     96        return p.lvalue instanceof Reference 
     97                && p.rvalue instanceof Reference; 
    9898    } 
    9999 
    …… 
    116116 
    117117        // single out main query and put join info. 
    118         List<SubQuery> joinedSubQueries = null; 
     118        List<SubQuery> joinedSubQueries; 
    119119        joinedSubQueries = new ArrayList<SubQuery>(subQueries.size()-1); 
    120120 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-search-compass-plugin/src/test/java/org/nuxeo/ecm/core/search/backend/compass/IntrospectableCompassBackend.java

    r21701 r28444  
    5555    } 
    5656 
    57     public Resource buildResource(CompassSession session, 
    58             ResolvedResource resource) 
    59         throws IndexingException { 
     57    public static Resource buildResource(CompassSession session, 
     58            ResolvedResource resource) throws IndexingException { 
    6059        return buildResource(session, resource, null, null, null, null); 
    6160    } 
    6261 
    6362    @SuppressWarnings("unchecked") 
    64     public Resource buildResource(CompassSession session, 
     63    public static Resource buildResource(CompassSession session, 
    6564            ResolvedResource resource, 
    6665            List<ResolvedData> commonData, 
    6766            String joinIdName, String joinIdValue, ACP acp) 
    6867            throws IndexingException { 
    69         return super.buildResource(session, null, resource, commonData, joinIdName, 
    70                 joinIdValue, acp); 
     68        return CompassBackend.buildResource(session, null, resource, commonData, 
     69                joinIdName, joinIdValue, acp); 
    7170    } 
    7271 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-search-compass-plugin/src/test/java/org/nuxeo/ecm/core/search/backend/compass/TestCompassBackendInternals.java

    r27078 r28444  
    458458            ResolvedResource iResource = SharedTestDataBuilder 
    459459                .aboutLifeIndexableBookSchemaResource(); 
    460             Resource r = backend.buildResource(session, iResource, 
     460            Resource r = IntrospectableCompassBackend.buildResource(session, iResource, 
    461461                    SharedTestDataBuilder.aboutLifeCommon(), 
    462462                    null, null, null); 
    …… 
    483483                .aboutLifeIndexableBookSchemaResource(); 
    484484 
    485             Resource r = backend.buildResource(session, null, iResource, 
     485            Resource r = IntrospectableCompassBackend.buildResource(session, null, iResource, 
    486486                    SharedTestDataBuilder.aboutLifeCommon(), 
    487487                    "join_id", 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-search-core-listener/src/test/java/org/nuxeo/ecm/core/search/api/security/SearchPolicyServiceImpl.java

    r24488 r28444  
    3434public class SearchPolicyServiceImpl implements SearchPolicyService { 
    3535 
    36     private static final Log log = LogFactory 
    37             .getLog(SearchPolicyServiceImpl.class); 
     36    private static final Log log = LogFactory.getLog(SearchPolicyServiceImpl.class); 
    3837 
    3938    public ComposedNXQuery applyPolicy(ComposedNXQuery nxqlQuery) { 
    …… 
    4443                Operator.LTEQ, new IntegerLiteral((Long) principal.getModel() 
    4544                        .getProperty("user", "accessLevel"))); 
    46         if (query.getWhereClause().toString().indexOf(add.toString()) < 0) { 
     45        if (!query.getWhereClause().toString().contains(add.toString())) { 
    4746            WhereClause wc; 
    4847            if (query.getWhereClause() != null) { 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-ui-web/src/main/java/org/nuxeo/ecm/platform/ui/web/directory/ChainSelect.java

    r26037 r28444  
    375375     */ 
    376376    public void rebuildOptions() { 
     377        /* 
    377378        for (int i = 0; i < size; i++) { 
    378379            if (optionList[i] != null) { 
    …… 
    383384            } 
    384385        } 
     386        */ 
    385387    } 
    386388 
    387389    private void rebuildOptions(int index) { 
    388  
    389390        ChainSelectListboxComponent component = getComponent(index); 
    390391 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-ui-web/src/main/java/org/nuxeo/ecm/platform/ui/web/util/ComponentTagUtils.java

    r19474 r28444  
    4141    public static boolean isValueReference(String value) { 
    4242        if (value == null) { 
    43             throw new NullPointerException(); 
     43            throw new IllegalArgumentException(); 
    4444        } 
    45         if (value.contains("#{") 
    46                 && (value.indexOf("#{") < value.indexOf('}'))) { 
    47             return true; 
    48         } 
    49         return false; 
     45        return value.contains("#{") 
     46                && value.indexOf("#{") < value.indexOf('}'); 
    5047    } 
    5148 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-usermanager-api/src/test/java/org/nuxeo/ecm/platform/usermanager/TestNuxeoPrincipalImpl.java

    r26508 r28444  
    5050        NuxeoPrincipalImpl a = new NuxeoPrincipalImpl("foo"); 
    5151        NuxeoPrincipalImpl b = new NuxeoPrincipalImpl("foo"); 
    52         assertTrue(a.hashCode() != 0); 
    5352        assertEquals(a.hashCode(), b.hashCode()); 
    5453        // we don't assert that hash codes are different for principals with 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-webapp-core/src/main/java/org/nuxeo/ecm/webapp/action/DeleteActionsBean.java

    r27893 r28444  
    267267        return checkDeletePermOnParents(docsToDelete); 
    268268    } 
    269      
     269 
    270270    public boolean getCanDeleteSections() throws ClientException { 
    271271        List<DocumentModel> docsToDelete = documentsListsManager 
    272272        .getWorkingList(DocumentsListsManager.CURRENT_DOCUMENT_SECTION_SELECTION); 
    273          
     273 
    274274        if (docsToDelete == null || docsToDelete.isEmpty()) { 
    275275            return false; 
    276276        } 
    277          
     277 
    278278        // do simple filtering 
    279279        return checkDeletePermOnParents(docsToDelete); 
    …… 
    356356                if (docToDelete.isLocked()) { 
    357357                    String locker = lockActions.getLockDetails(docToDelete) 
    358                             .get(lockActions.LOCKER); 
     358                            .get(LockActions.LOCKER); 
    359359                    if (currentUser.getName().equals(locker)) { 
    360360                        docsThatCanBeDeleted.add(docToDelete); 
    …… 
    393393                    .getWorkingList(DocumentsListsManager.CURRENT_DOCUMENT_SELECTION)); 
    394394        } else { 
    395             log 
    396                     .debug("No documents selection in context to process delete on..."); 
     395            log.debug("No documents selection in context to process delete on..."); 
    397396            return null; 
    398397        } 
    399398    } 
    400      
     399 
    401400    public String deleteSelectionSections() throws ClientException { 
    402401        if (!documentsListsManager 
    …… 
    405404                    .getWorkingList(DocumentsListsManager.CURRENT_DOCUMENT_SECTION_SELECTION)); 
    406405        } else { 
    407             log 
    408             .debug("No sections selection in context to process delete on..."); 
     406            log.debug("No sections selection in context to process delete on..."); 
    409407            return null; 
    410408        } 
    …… 
    471469 
    472470            // Notify parents 
    473             DocumentModel parent; 
    474471            for (DocumentRef parentRef : parentsRefs) { 
    475                 parent = documentManager.getDocument(parentRef); 
     472                DocumentModel parent = documentManager.getDocument(parentRef); 
    476473                if (parent != null) { 
    477474                    Events.instance().raiseEvent( 
    …… 
    554551                    .getWorkingList(DocumentsListsManager.CURRENT_DOCUMENT_TRASH_SELECTION)); 
    555552        } else { 
    556             log 
    557                     .debug("No documents selection in context to process delete on..."); 
     553            log.debug("No documents selection in context to process delete on..."); 
    558554            return null; 
    559555        } 
    …… 
    639635 
    640636        return computeOutcome(DELETE_OUTCOME); 
    641  
    642637    } 
    643638 
    …… 
    695690            } 
    696691        } 
    697  
    698692    } 
    699693 
    …… 
    753747                    data); 
    754748        } 
    755  
    756749    } 
    757750 
    …… 
    779772            SortInfo sortInfo) throws ClientException, 
    780773            ResultsProviderFarmUserException { 
    781         Object[] params = null; 
     774        Object[] params; 
    782775        PagedDocumentsProvider provider = null; 
    783776 
    …… 
    890883 
    891884    public boolean getCanRestoreCurrentDoc() throws ClientException{ 
    892         if ("deleted".equals(navigationContext.getCurrentDocument().getCurrentLifeCycleState())){ 
    893             return true; 
    894         } 
    895         return false; 
     885        return "deleted".equals( 
     886                navigationContext.getCurrentDocument().getCurrentLifeCycleState()); 
    896887    } 
    897888 

  • org.nuxeo.ecm.platform/trunk/nuxeo-platform-webapp-core/src/main/java/org/nuxeo/ecm/webapp/clipboard/ClipboardActionsBean.java

    r27152 r28444  
    801801 
    802802    public void setCurrentSelectedList(String listId) { 
    803         if ((listId != null) && (!listId.equals(currentSelectedList))) { 
     803        if (listId != null && !listId.equals(currentSelectedList)) { 
    804804            currentSelectedList = listId; 
    805805            canEditSelectedDocs = null; 
    …… 
    892892    public boolean getCanEditSelectedDocs() throws ClientException { 
    893893        if (canEditSelectedDocs == null) { 
    894             if (!getCurrentSelectedList().isEmpty()) { 
     894            if (getCurrentSelectedList().isEmpty()) { 
     895                canEditSelectedDocs = false; 
     896            } else { 
    895897                final List<DocumentModel> selectedDocs = getCurrentSelectedList(); 
    896898 
    897899                // check selected docs 
    898900                canEditSelectedDocs = checkWritePerm(selectedDocs); 
    899             } else { 
    900                 canEditSelectedDocs = false; 
    901901            } 
    902902        } 
    …… 
    908908 
    909909        final boolean canEdit; 
    910         if (!docs.isEmpty()) { 
     910        if (docs.isEmpty()) { 
     911            canEdit = false; 
     912        } else { 
    911913            // check selected docs 
    912914            canEdit = checkWritePerm(docs); 
    913         } else { 
    914             canEdit = false; 
    915915        }