MinnPost/object-sync-for-salesforce

View on GitHub
classes/class-object-sync-sf-salesforce-pull.php

Summary

Maintainability
F
2 wks
Test Coverage

File class-object-sync-sf-salesforce-pull.php has 1376 lines of code (exceeds 250 allowed). Consider refactoring.
Open

<?php
/**
 * Pull data from Salesforce into WordPress
 *
 * @class   Object_Sync_Sf_Salesforce_Pull
Severity: Major
Found in classes/class-object-sync-sf-salesforce-pull.php - About 3 days to fix

    Function salesforce_pull_process_records has a Cognitive Complexity of 99 (exceeds 5 allowed). Consider refactoring.
    Open

        public function salesforce_pull_process_records( $object_type, $object, $sf_sync_trigger ) {
    
            $sfapi = $this->salesforce['sfapi'];
    
            if ( is_string( $object ) ) {
    Severity: Minor
    Found in classes/class-object-sync-sf-salesforce-pull.php - About 1 day to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Function get_updated_records has a Cognitive Complexity of 78 (exceeds 5 allowed). Consider refactoring.
    Open

        private function get_updated_records() {
            $sfapi = $this->salesforce['sfapi'];
            foreach ( $this->mappings->get_fieldmaps( null, $this->mappings->active_fieldmap_conditions ) as $salesforce_mapping ) {
                $map_sync_triggers = $salesforce_mapping['sync_triggers']; // this sets which Salesforce triggers are allowed for the mapping.
                $type              = $salesforce_mapping['salesforce_object']; // this sets the Salesforce object type for the SOQL query.
    Severity: Minor
    Found in classes/class-object-sync-sf-salesforce-pull.php - About 1 day to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method salesforce_pull_process_records has 239 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

        public function salesforce_pull_process_records( $object_type, $object, $sf_sync_trigger ) {
    
            $sfapi = $this->salesforce['sfapi'];
    
            if ( is_string( $object ) ) {
    Severity: Major
    Found in classes/class-object-sync-sf-salesforce-pull.php - About 1 day to fix

      Method create_called_from_salesforce has 215 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

          private function create_called_from_salesforce( $sf_sync_trigger, $synced_object, $params, $prematch, $wordpress_id_field_name, $seconds ) {
      
              $salesforce_mapping = $synced_object['mapping'];
              $object             = $synced_object['salesforce_object'];
              // methods to run the wp update operations.
      Severity: Major
      Found in classes/class-object-sync-sf-salesforce-pull.php - About 1 day to fix

        Method get_updated_records has 213 lines of code (exceeds 25 allowed). Consider refactoring.
        Open

            private function get_updated_records() {
                $sfapi = $this->salesforce['sfapi'];
                foreach ( $this->mappings->get_fieldmaps( null, $this->mappings->active_fieldmap_conditions ) as $salesforce_mapping ) {
                    $map_sync_triggers = $salesforce_mapping['sync_triggers']; // this sets which Salesforce triggers are allowed for the mapping.
                    $type              = $salesforce_mapping['salesforce_object']; // this sets the Salesforce object type for the SOQL query.
        Severity: Major
        Found in classes/class-object-sync-sf-salesforce-pull.php - About 1 day to fix

          Function create_called_from_salesforce has a Cognitive Complexity of 52 (exceeds 5 allowed). Consider refactoring.
          Open

              private function create_called_from_salesforce( $sf_sync_trigger, $synced_object, $params, $prematch, $wordpress_id_field_name, $seconds ) {
          
                  $salesforce_mapping = $synced_object['mapping'];
                  $object             = $synced_object['salesforce_object'];
                  // methods to run the wp update operations.
          Severity: Minor
          Found in classes/class-object-sync-sf-salesforce-pull.php - About 1 day to fix

          Cognitive Complexity

          Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

          A method's cognitive complexity is based on a few simple rules:

          • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
          • Code is considered more complex for each "break in the linear flow of the code"
          • Code is considered more complex when "flow breaking structures are nested"

          Further reading

          Function get_deleted_records has a Cognitive Complexity of 49 (exceeds 5 allowed). Consider refactoring.
          Open

              private function get_deleted_records() {
          
                  $sfapi = $this->salesforce['sfapi'];
          
                  // Load all unique SF record types that we have mappings for. This results in a double loop.
          Severity: Minor
          Found in classes/class-object-sync-sf-salesforce-pull.php - About 7 hrs to fix

          Cognitive Complexity

          Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

          A method's cognitive complexity is based on a few simple rules:

          • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
          • Code is considered more complex for each "break in the linear flow of the code"
          • Code is considered more complex when "flow breaking structures are nested"

          Further reading

          Function get_merged_records has a Cognitive Complexity of 38 (exceeds 5 allowed). Consider refactoring.
          Open

              private function get_merged_records() {
          
                  $use_soap = $this->salesforce['soap_loaded'];
                  if ( true === $use_soap ) {
                      $soap = new Object_Sync_Sf_Salesforce_Soap_Partner();
          Severity: Minor
          Found in classes/class-object-sync-sf-salesforce-pull.php - About 5 hrs to fix

          Cognitive Complexity

          Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

          A method's cognitive complexity is based on a few simple rules:

          • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
          • Code is considered more complex for each "break in the linear flow of the code"
          • Code is considered more complex when "flow breaking structures are nested"

          Further reading

          Method delete_called_from_salesforce has 95 lines of code (exceeds 25 allowed). Consider refactoring.
          Open

              private function delete_called_from_salesforce( $sf_sync_trigger, $synced_object, $wordpress_id_field_name, $seconds, $mapping_objects ) {
          
                  $salesforce_mapping = $synced_object['mapping'];
                  $mapping_object     = $synced_object['mapping_object'];
          
          
          Severity: Major
          Found in classes/class-object-sync-sf-salesforce-pull.php - About 3 hrs to fix

            Object_Sync_Sf_Salesforce_Pull has 29 functions (exceeds 20 allowed). Consider refactoring.
            Open

            class Object_Sync_Sf_Salesforce_Pull {
            
                /**
                 * Current version of the plugin
                 *
            Severity: Minor
            Found in classes/class-object-sync-sf-salesforce-pull.php - About 3 hrs to fix

              Method get_deleted_records has 83 lines of code (exceeds 25 allowed). Consider refactoring.
              Open

                  private function get_deleted_records() {
              
                      $sfapi = $this->salesforce['sfapi'];
              
                      // Load all unique SF record types that we have mappings for. This results in a double loop.
              Severity: Major
              Found in classes/class-object-sync-sf-salesforce-pull.php - About 3 hrs to fix

                Method get_pull_query has 74 lines of code (exceeds 25 allowed). Consider refactoring.
                Open

                    private function get_pull_query( $type, $salesforce_mapping = array() ) {
                        // we need to determine what to do with saved queries. this is what we currently do but it doesn't work.
                        // check if we have a stored next query to run for this type. if so, unserialize it so we have an object.
                        $pull_query_running = $this->pull_options->get( 'current_query', $type, $salesforce_mapping['id'], '' );
                        if ( '' !== $pull_query_running ) {
                Severity: Major
                Found in classes/class-object-sync-sf-salesforce-pull.php - About 2 hrs to fix

                  Function delete_called_from_salesforce has a Cognitive Complexity of 21 (exceeds 5 allowed). Consider refactoring.
                  Open

                      private function delete_called_from_salesforce( $sf_sync_trigger, $synced_object, $wordpress_id_field_name, $seconds, $mapping_objects ) {
                  
                          $salesforce_mapping = $synced_object['mapping'];
                          $mapping_object     = $synced_object['mapping_object'];
                  
                  
                  Severity: Minor
                  Found in classes/class-object-sync-sf-salesforce-pull.php - About 2 hrs to fix

                  Cognitive Complexity

                  Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                  A method's cognitive complexity is based on a few simple rules:

                  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                  • Code is considered more complex for each "break in the linear flow of the code"
                  • Code is considered more complex when "flow breaking structures are nested"

                  Further reading

                  Method update_called_from_salesforce has 68 lines of code (exceeds 25 allowed). Consider refactoring.
                  Open

                      private function update_called_from_salesforce( $sf_sync_trigger, $synced_object, $params, $wordpress_id_field_name, $seconds ) {
                  
                          $salesforce_mapping = $synced_object['mapping'];
                          $mapping_object     = $synced_object['mapping_object'];
                          $object             = $synced_object['salesforce_object'];
                  Severity: Major
                  Found in classes/class-object-sync-sf-salesforce-pull.php - About 2 hrs to fix

                    Method get_merged_records has 60 lines of code (exceeds 25 allowed). Consider refactoring.
                    Open

                        private function get_merged_records() {
                    
                            $use_soap = $this->salesforce['soap_loaded'];
                            if ( true === $use_soap ) {
                                $soap = new Object_Sync_Sf_Salesforce_Soap_Partner();
                    Severity: Major
                    Found in classes/class-object-sync-sf-salesforce-pull.php - About 2 hrs to fix

                      Function get_pull_query has a Cognitive Complexity of 17 (exceeds 5 allowed). Consider refactoring.
                      Open

                          private function get_pull_query( $type, $salesforce_mapping = array() ) {
                              // we need to determine what to do with saved queries. this is what we currently do but it doesn't work.
                              // check if we have a stored next query to run for this type. if so, unserialize it so we have an object.
                              $pull_query_running = $this->pull_options->get( 'current_query', $type, $salesforce_mapping['id'], '' );
                              if ( '' !== $pull_query_running ) {
                      Severity: Minor
                      Found in classes/class-object-sync-sf-salesforce-pull.php - About 2 hrs to fix

                      Cognitive Complexity

                      Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                      A method's cognitive complexity is based on a few simple rules:

                      • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                      • Code is considered more complex for each "break in the linear flow of the code"
                      • Code is considered more complex when "flow breaking structures are nested"

                      Further reading

                      Function get_next_record_batch has a Cognitive Complexity of 16 (exceeds 5 allowed). Consider refactoring.
                      Open

                          private function get_next_record_batch( $last_sync, $salesforce_mapping, $map_sync_triggers, $type, $version_path, $query_options, $response ) {
                              // Handle next batch of records if it exists.
                              $next_records_url = isset( $response['nextRecordsUrl'] ) ? str_replace( $version_path, '', $response['nextRecordsUrl'] ) : false;
                              while ( $next_records_url ) {
                                  // shouldn't cache this either. it's going into the queue if it exists anyway.
                      Severity: Minor
                      Found in classes/class-object-sync-sf-salesforce-pull.php - About 2 hrs to fix

                      Cognitive Complexity

                      Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                      A method's cognitive complexity is based on a few simple rules:

                      • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                      • Code is considered more complex for each "break in the linear flow of the code"
                      • Code is considered more complex when "flow breaking structures are nested"

                      Further reading

                      Method get_next_record_batch has 38 lines of code (exceeds 25 allowed). Consider refactoring.
                      Open

                          private function get_next_record_batch( $last_sync, $salesforce_mapping, $map_sync_triggers, $type, $version_path, $query_options, $response ) {
                              // Handle next batch of records if it exists.
                              $next_records_url = isset( $response['nextRecordsUrl'] ) ? str_replace( $version_path, '', $response['nextRecordsUrl'] ) : false;
                              while ( $next_records_url ) {
                                  // shouldn't cache this either. it's going into the queue if it exists anyway.
                      Severity: Minor
                      Found in classes/class-object-sync-sf-salesforce-pull.php - About 1 hr to fix

                        Method respond_to_salesforce_merge has 32 lines of code (exceeds 25 allowed). Consider refactoring.
                        Open

                            private function respond_to_salesforce_merge( $object_type, $merged_record ) {
                                $op = 'Merge';
                                if ( isset( $merged_record['Id'] ) && true === filter_var( $merged_record['sf:IsDeleted'], FILTER_VALIDATE_BOOLEAN ) && '' !== $merged_record['sf:MasterRecordId'] ) {
                                    $previous_sf_id  = $merged_record['Id'];
                                    $new_sf_id       = $merged_record['sf:MasterRecordId'];
                        Severity: Minor
                        Found in classes/class-object-sync-sf-salesforce-pull.php - About 1 hr to fix

                          Method get_next_record_batch has 7 arguments (exceeds 4 allowed). Consider refactoring.
                          Open

                              private function get_next_record_batch( $last_sync, $salesforce_mapping, $map_sync_triggers, $type, $version_path, $query_options, $response ) {
                          Severity: Major
                          Found in classes/class-object-sync-sf-salesforce-pull.php - About 50 mins to fix

                            Method create_called_from_salesforce has 6 arguments (exceeds 4 allowed). Consider refactoring.
                            Open

                                private function create_called_from_salesforce( $sf_sync_trigger, $synced_object, $params, $prematch, $wordpress_id_field_name, $seconds ) {
                            Severity: Minor
                            Found in classes/class-object-sync-sf-salesforce-pull.php - About 45 mins to fix

                              Method update_called_from_salesforce has 5 arguments (exceeds 4 allowed). Consider refactoring.
                              Open

                                  private function update_called_from_salesforce( $sf_sync_trigger, $synced_object, $params, $wordpress_id_field_name, $seconds ) {
                              Severity: Minor
                              Found in classes/class-object-sync-sf-salesforce-pull.php - About 35 mins to fix

                                Method is_pull_allowed has 5 arguments (exceeds 4 allowed). Consider refactoring.
                                Open

                                    private function is_pull_allowed( $object_type, $object, $sf_sync_trigger, $salesforce_mapping, $map_sync_triggers ) {
                                Severity: Minor
                                Found in classes/class-object-sync-sf-salesforce-pull.php - About 35 mins to fix

                                  Method delete_called_from_salesforce has 5 arguments (exceeds 4 allowed). Consider refactoring.
                                  Open

                                      private function delete_called_from_salesforce( $sf_sync_trigger, $synced_object, $wordpress_id_field_name, $seconds, $mapping_objects ) {
                                  Severity: Minor
                                  Found in classes/class-object-sync-sf-salesforce-pull.php - About 35 mins to fix

                                    Function manual_pull has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
                                    Open

                                        public function manual_pull( $object_type, $salesforce_id = '' ) {
                                    
                                            if ( '' === $salesforce_id ) {
                                                $sf_sync_trigger = $this->mappings->sync_sf_create;
                                            } else {
                                    Severity: Minor
                                    Found in classes/class-object-sync-sf-salesforce-pull.php - About 25 mins to fix

                                    Cognitive Complexity

                                    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                                    A method's cognitive complexity is based on a few simple rules:

                                    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                                    • Code is considered more complex for each "break in the linear flow of the code"
                                    • Code is considered more complex when "flow breaking structures are nested"

                                    Further reading

                                    Similar blocks of code found in 2 locations. Consider refactoring.
                                    Open

                                                if ( 1 === $salesforce_pushing ) {
                                                    $transients_to_delete[ $fieldmap_key ]['transients'][] = $mapping_object_id_transient;
                                                    if ( true === $this->debug ) {
                                                        // create log entry for failed pull.
                                                        $status = 'debug';
                                    Severity: Major
                                    Found in classes/class-object-sync-sf-salesforce-pull.php and 1 other location - About 2 hrs to fix
                                    classes/class-object-sync-sf-salesforce-push.php on lines 573..597

                                    Duplicated Code

                                    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                    Tuning

                                    This issue has a mass of 136.

                                    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                    Refactorings

                                    Further Reading

                                    Similar blocks of code found in 2 locations. Consider refactoring.
                                    Open

                                                            if ( true === $this->debug ) {
                                                                // create log entry for successful pull.
                                                                $status = 'debug';
                                                                $title  = sprintf(
                                                                    // translators: placeholders are: 1) the log status, 2) the Salesforce ID.
                                    Severity: Major
                                    Found in classes/class-object-sync-sf-salesforce-pull.php and 1 other location - About 2 hrs to fix
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 456..473

                                    Duplicated Code

                                    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                    Tuning

                                    This issue has a mass of 130.

                                    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                    Refactorings

                                    Further Reading

                                    Similar blocks of code found in 2 locations. Consider refactoring.
                                    Open

                                                                if ( true === $this->debug ) {
                                                                    // create log entry for failed pull.
                                                                    $status = 'debug';
                                                                    $title  = sprintf(
                                                                        // translators: placeholders are: 1) the log status, 2) the Salesforce ID.
                                    Severity: Major
                                    Found in classes/class-object-sync-sf-salesforce-pull.php and 1 other location - About 2 hrs to fix
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 517..534

                                    Duplicated Code

                                    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                    Tuning

                                    This issue has a mass of 130.

                                    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                    Refactorings

                                    Further Reading

                                    Similar blocks of code found in 2 locations. Consider refactoring.
                                    Open

                                            foreach ( $transients_to_delete as $key => $value ) {
                                                $fieldmap_id = $value['fieldmap']['id'];
                                                $transients  = $value['transients'];
                                                foreach ( $transients as $transient_end ) {
                                                    $this->sync_transients->delete( 'salesforce_pushing_' . $transient_end, '', $fieldmap_id );
                                    Severity: Major
                                    Found in classes/class-object-sync-sf-salesforce-pull.php and 1 other location - About 1 hr to fix
                                    classes/class-object-sync-sf-salesforce-push.php on lines 747..757

                                    Duplicated Code

                                    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                    Tuning

                                    This issue has a mass of 120.

                                    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                    Refactorings

                                    Further Reading

                                    Similar blocks of code found in 5 locations. Consider refactoring.
                                    Open

                                                            $title  = sprintf(
                                                                // translators: placeholders are: 1) the log status, 2) what operation is happening, 3) the name of the WordPress object type, 4) the WordPress id field name, 5) the WordPress object id value, 6) the name of the Salesforce object, 7) the Salesforce Id value.
                                                                esc_html__( '%1$s: %2$s WordPress %3$s with %4$s of %5$s (%6$s %7$s)', 'object-sync-for-salesforce' ),
                                                                ucfirst( esc_attr( $status ) ),
                                                                esc_attr( $op ),
                                    Severity: Major
                                    Found in classes/class-object-sync-sf-salesforce-pull.php and 4 other locations - About 1 hr to fix
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 1952..1962
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 1981..1991
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2091..2101
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2139..2149

                                    Duplicated Code

                                    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                    Tuning

                                    This issue has a mass of 117.

                                    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                    Refactorings

                                    Further Reading

                                    Similar blocks of code found in 5 locations. Consider refactoring.
                                    Open

                                                $title  = sprintf(
                                                    // translators: placeholders are: 1) the log status, 2) what operation is happening, 3) the name of the WordPress object, 4) the WordPress id field name, 5) the WordPress object id value, 6) the name of the Salesforce object, 7) the Salesforce Id value.
                                                    esc_html__( '%1$s: %2$s WordPress %3$s with %4$s of %5$s (Salesforce %6$s with Id of %7$s)', 'object-sync-for-salesforce' ),
                                                    ucfirst( esc_attr( $status ) ),
                                                    esc_attr( $op ),
                                    Severity: Major
                                    Found in classes/class-object-sync-sf-salesforce-pull.php and 4 other locations - About 1 hr to fix
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 1952..1962
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2067..2077
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2091..2101
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2139..2149

                                    Duplicated Code

                                    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                    Tuning

                                    This issue has a mass of 117.

                                    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                    Refactorings

                                    Further Reading

                                    Similar blocks of code found in 5 locations. Consider refactoring.
                                    Open

                                                        $title  = sprintf(
                                                            // translators: placeholders are: 1) the operation that is happening, 2) the name of the WordPress object type, 3) the WordPress id field name, 4) the WordPress object id value, 5) the name of the Salesforce object type, 6) the Salesforce Id.
                                                            esc_html__( '%1$s: %2$s on WordPress %3$s with %4$s of %5$s was stopped because there are other WordPress records mapped to Salesforce %6$s of %7$s', 'object-sync-for-salesforce' ),
                                                            ucfirst( esc_attr( $status ) ),
                                                            esc_attr( $op ),
                                    Severity: Major
                                    Found in classes/class-object-sync-sf-salesforce-pull.php and 4 other locations - About 1 hr to fix
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 1952..1962
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 1981..1991
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2067..2077
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2091..2101

                                    Duplicated Code

                                    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                    Tuning

                                    This issue has a mass of 117.

                                    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                    Refactorings

                                    Further Reading

                                    Similar blocks of code found in 5 locations. Consider refactoring.
                                    Open

                                                $title  = sprintf(
                                                    // translators: placeholders are: 1) the log status, 2) what operation is happening, 3) the name of the WordPress object type, 4) the WordPress id field name, 5) the WordPress object id value, 6) the name of the Salesforce object, 7) the Salesforce Id value.
                                                    esc_html__( '%1$s: %2$s WordPress %3$s with %4$s of %5$s (Salesforce %6$s Id of %7$s)', 'object-sync-for-salesforce' ),
                                                    ucfirst( esc_attr( $status ) ),
                                                    esc_attr( $op ),
                                    Severity: Major
                                    Found in classes/class-object-sync-sf-salesforce-pull.php and 4 other locations - About 1 hr to fix
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 1981..1991
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2067..2077
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2091..2101
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2139..2149

                                    Duplicated Code

                                    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                    Tuning

                                    This issue has a mass of 117.

                                    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                    Refactorings

                                    Further Reading

                                    Similar blocks of code found in 5 locations. Consider refactoring.
                                    Open

                                                            $title = sprintf(
                                                                // translators: placeholders are: 1) the log status, 2) what operation is happening, 3) the name of the WordPress object type, 4) the WordPress id field name, 5) the WordPress object id value, 6) the name of the Salesforce object, 7) the Salesforce Id value.
                                                                esc_html__( '%1$s: %2$s WordPress %3$s with %4$s of %5$s (%6$s %7$s)', 'object-sync-for-salesforce' ),
                                                                ucfirst( esc_attr( $status ) ),
                                                                esc_attr( $op ),
                                    Severity: Major
                                    Found in classes/class-object-sync-sf-salesforce-pull.php and 4 other locations - About 1 hr to fix
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 1952..1962
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 1981..1991
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2067..2077
                                    classes/class-object-sync-sf-salesforce-pull.php on lines 2139..2149

                                    Duplicated Code

                                    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                    Tuning

                                    This issue has a mass of 117.

                                    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                    Refactorings

                                    Further Reading

                                    There are no issues that match your filters.

                                    Category
                                    Status