Answers

• allan Posts: 61,650Questions: 1Answers: 10,094 Site admin

  September 2020 Answer ✓

  Hi Colin,

  The error message is correct - there is no $data['id'] as you can see from the var_dump of the $data parameter. The $data parameter contains the raw data submitted by the client which could be multiple rows! Also the primary key value is in the parameter name for the $data['data'] object.

  So what you need to do is loop over $data['data']:

  foreach ( $data['data'] as $pkey => $values ) {
  

  That is shown in the second example here.

  You'll also need to use str_replace to remove the row_ prefix from the pkey value if you want to query the database with it.

  Allan
• cpshart Posts: 246Questions: 49Answers: 5

  September 2020

  Hi Allan

  Thanks for your advise, I will do some more background reading as suggested, before amending the code, I will get back to you with my results.

  Best Regards

  Colin
• cpshart Posts: 246Questions: 49Answers: 5

  September 2020

  Hi Allan

  Thanks again, all is now working, I have posted some key code extracts below.

  server file extract issuing an error on the modal if there are holding lines defined for the parent holdings

      ->validator( function ( $editor, $action, $data ) use ($userid) { //1
          global $count;
  
          if ( $action === Editor::ACTION_EDIT ) { //2
             foreach ( $data['data'] as $pkey => $values ) { //3
  
                      $count = $editor
                              ->db()
                              ->query('select')
                              ->get('*')
                              ->table('dm_holdinglines')
                              ->where( function ( $q ) use ( $userid) {
                              $q->where( 'user_id', $userid);
                              } )
                              ->and_where( 'holdings_id', $values['dm_holdings']['id'] )
                              ->exec()
                              ->count();
                             
                      if ($count > 0){ //4
                          return 'cannot edit holdings with ' . $count . ' holding lines defined';
                      } //4
  
             } //3
  
          } //2
      } ) //1
  
  

  client file extract, passing the id to the server which is hidden, so not visible to users to edit on the modal

          table: "#dm_holdings",
          fields: [ {
                  label: "id:",
                  name: "dm_holdings.id",         
                  type: "hidden"
              }, {
  
  

  Best Regards

  Colin
• cpshart Posts: 246Questions: 49Answers: 5

  September 2020

  Hi Allan

  I am having a major issue with passing the dm_holdings.id being the primary key from the client as shown above, I can edit the table no problems, but I cannot create a New Row, as it is not allocating the next id (defined as AUTO INCREMENT in the MySQL Table)

  client extract

      var siteEditor = new $.fn.dataTable.Editor({
          ajax: {
              url: "../../" + EDITOR_DIR + "/controllers/ukif-stock_holdings.php",
              type: 'POST',
              data: function ( d ) {
                  d.userid = $('#passuserid').val();
              }
          },
          table: "#dm_holdings",
          fields: [ {
                  label: "id:",
          name: "dm_holdings.id",        
          type: "hidden"
              }, {
                  label: "portfolio:",
                  name: "dm_holdings.portfolio_id",
                           type: "select2",
         placeholder: "Select a portfolio"
              }, {
  
  

  For testing I have removed the type: "hidden" and the id field is empty when adding a new row to the table.

  extract of server file

  // Build our Editor instance and process the data coming from _POST
  Editor::inst( $db, 'dm_holdings' )
  
      ->fields( 
          Field::inst( 'dm_holdings.id' ),
          Field::inst( 'dm_holdings.user_id' )
              ->setValue( $userid ),            
  
          Field::inst( 'dm_holdings.portfolio_id' )
              ->options( Options::inst()
                  ->table( 'dm_portfolios' )
                  ->value( 'id' )
                  ->label( array('code', 'name') )
                  ->where( function ( $q ) use ( $userid) {
                      $q
                      ->where( 'dm_portfolios.user_id', $userid )
                      ->and_where('dm_portfolios.reporting_status', 'yes');
                  } )
              )
              ->validator( Validate::notEmpty( ValidateOptions::inst()
                  ->message( 'Please select portfolio ..' )
              ) ),    
          Field::inst( 'dm_portfolios.code' ),
  
  

  
  

  My question is how can I force the population of the dm_holdings.id value on adding a New Row to the next available id normally done automatically.

  If I remove the passing of id in the client, i.e. the code below I can add a New record and it would auto increment the id to its correct value in the MySQL Table

  {
                  label: "id:",
          name: "dm_holdings.id",        
          type: "hidden"
              }, 
  

  But I need to pass the id in order to check for holding lines and perform extra processing accordingly, most of which is now working !.

  As before let me know if you need access to my system, I can provide the links to files etc.

  Thanks in advance

  Colin
• cpshart Posts: 246Questions: 49Answers: 5

  September 2020

  Hi Allan

  Subject to more testing, I think I have fixed the problem, I needed to add ->set(false) to the server file as follows

  Editor::inst( $db, 'dm_holdings' )
  
      ->fields( 
          Field::inst( 'dm_holdings.id' )->set(false),
  
  

  Many Thanks

  Colin