Branch: refs/heads/master
  Home:   https://github.com/pmd/pmd
  Commit: 3735fd145b1385c914bcea1ac363866e33cc7ede
      https://github.com/pmd/pmd/commit/3735fd145b1385c914bcea1ac363866e33cc7ede
  Author: Andreas Dangel <andreas.dan...@pmd-code.org>
  Date:   2024-07-27 (Sat, 27 Jul 2024)

  Changed paths:
    M docs/pages/release_notes.md
    M pmd-apex/src/main/java/net/sourceforge/pmd/lang/apex/ast/ASTMethod.java
    M 
pmd-apex/src/main/java/net/sourceforge/pmd/lang/apex/ast/ApexTreeBuilder.kt
    M 
pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexTreeDumpTest.java
    A 
pmd-apex/src/test/resources/net/sourceforge/pmd/lang/apex/ast/AccountTrigger.cls
    A 
pmd-apex/src/test/resources/net/sourceforge/pmd/lang/apex/ast/AccountTrigger.txt
    M 
pmd-apex/src/test/resources/net/sourceforge/pmd/lang/apex/rule/design/xml/CyclomaticComplexity.xml
    M 
pmd-apex/src/test/resources/net/sourceforge/pmd/lang/apex/rule/performance/xml/OperationWithHighCostInLoop.xml

  Log Message:
  -----------
  [apex] Fix parsing of triggers with declarations

Only the grandchildren of a trigger block were ending up in the tree,
but the direct children of triggerBlock were missing, e.g.
ForLoopStatement. This caused OperationWithHighCostInLoop to not
find the loop anymore in triggers.

This will probably fix other false negatives in triggers in
other rules as well.

Fixes #5139


  Commit: 9ea10ecbe3b44885c5b696ddac4c157117f5ca0e
      https://github.com/pmd/pmd/commit/9ea10ecbe3b44885c5b696ddac4c157117f5ca0e
  Author: Juan Martín Sotuyo Dodero <juansot...@gmail.com>
  Date:   2024-07-27 (Sat, 27 Jul 2024)

  Changed paths:
    M docs/pages/release_notes.md
    M pmd-apex/src/main/java/net/sourceforge/pmd/lang/apex/ast/ASTMethod.java
    M 
pmd-apex/src/main/java/net/sourceforge/pmd/lang/apex/ast/ApexTreeBuilder.kt
    M 
pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexTreeDumpTest.java
    A 
pmd-apex/src/test/resources/net/sourceforge/pmd/lang/apex/ast/AccountTrigger.cls
    A 
pmd-apex/src/test/resources/net/sourceforge/pmd/lang/apex/ast/AccountTrigger.txt
    M 
pmd-apex/src/test/resources/net/sourceforge/pmd/lang/apex/rule/design/xml/CyclomaticComplexity.xml
    M 
pmd-apex/src/test/resources/net/sourceforge/pmd/lang/apex/rule/performance/xml/OperationWithHighCostInLoop.xml

  Log Message:
  -----------
  Merge pull request #5146 from 
adangel/issue-5139-apex-operationwithhighcostinloop

[apex] Fix parsing of triggers with declarations


Compare: https://github.com/pmd/pmd/compare/d2fbe14afaf2...9ea10ecbe3b4

To unsubscribe from these emails, change your notification settings at 
https://github.com/pmd/pmd/settings/notifications


_______________________________________________
Pmd-commits mailing list
Pmd-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/pmd-commits

Reply via email to