Does formatFormValues load form data?

[flashpoint]

Hi all,

I've been trying to make my own plugin that creates threads automatically when a new gallery image or album is posted for the last few days but I am stuck on the form for each category. For whatever reason that data does not load the data back onto the form but it does save onto the database. 

This is what I have for formatFormValues:

	public function formatFormValues( $values )
	{
	
			$values = parent::formatFormValues( $values );
				
	
			if ( $this->id )
			{
				$save = array( 'threadgen_gallery_forum_on' => (int)$values['threadgen_gallery_forum_on'], 'threadgen_gallery_category_id' => $this->id, 'threadgen_gallery_forum_id' => $values['threadgen_gallery_forum_id']->id, 'threadgen_gallery_topic_prefix' => (string)$values['threadgen_gallery_topic_prefix'], 'threadgen_gallery_topic_suffix' => (string)$values['threadgen_gallery_topic_suffix'], 'threadgen_gallery_topic_delete' => (int)$values['threadgen_gallery_topic_delete'] );
				foreach ( $values as $k => $v )
				{
					if ( \in_array( $k, array( 'threadgen_gallery_category_id', 'threadgen_gallery_forum_id', 'threadgen_gallery_topic_prefix', 'threadgen_gallery_topic_suffix', 'threadgen_gallery_topic_delete' ) ) )
					{
						unset( $values[$k] );
					}
				}
				
	
				if ( \count( $save ) )
				{
					\IPS\Db::i()->insert( 'gallery_threadgen_addon', $save, TRUE );
				}
			}
			return $values;
		
	}

Basically the question is does formatFormValues load the values back or only save them? If it only saves them how would I load them back?

Thanks for your time and help! 🙂

Second parameter for forms is default value whoops 😅

Edited October 25, 2021 by flashpoint

[Daniel F]

After seeing 

gallery_threadgen_addon

I have a minor suggestion => Please use your own database table prefix! It's going to be much easier for you (but also for our staff to not mix up any 3rd party database tables with our own!

We have also covered this in our guidelines

1 hour ago, flashpoint said:

Second parameter for forms is default value whoops 😅

YEP;)

[flashpoint]

30 minutes ago, Daniel F said:

After seeing 

gallery_threadgen_addon

I have a minor suggestion => Please use your own database table prefix! It's going to be much easier for you (but also for our staff to not mix up any 3rd party database tables with our own!

We have also covered this in our guidelines

YEP;)

Will do! Still having issues loading the saved data from the database though

    public function save()

    {

        try
        {
            $data = $this->_data;
\IPS\Log::log( $data, 'save function threadgen'  );
            $joinTableData = array();
\IPS\Log::log( $joinTableData, 'save function threadgen data'  );
\IPS\Log::log( $databasePrefix, 'thread_gen_db_prefix'  );
            foreach ( ( $this->_new ? $this->_data : $this->changed ) as $k => $v )
            {

                if ( \in_array( $k, array('threadgen_gallery_forum_on', 'gallery_forum_on', 'threadgen_gallery_category_id', 'threadgen_gallery_forum_id', 'threadgen_gallery_topic_prefix', 'threadgen_gallery_topic_suffix', 'threadgen_gallery_topic_delete'  ) ) )
                {
\IPS\Log::log( $k . $v, 'save function threadgen foreach'  );
                    $joinTableData[ static::$databasePrefix . $k ] = $v;
                    unset( $this->_data[ $k ] );
                    unset( $this->changed[ $k ] );
                }
            }


    
            parent::save();
    
            $data['id'] = $this->_data['id'];
            $this->_data = $data;
    
            if ( \count( $joinTableData ) )
            {
                $joinTableData['threadgen_gallery_category_id'] = $this->id;
    
                \IPS\Db::i()->insert( 'autothreadgen_gallery_addon', $joinTableData, TRUE );
            }
        }
        catch ( \RuntimeException $e )
        {
            if ( method_exists( get_parent_class(), __FUNCTION__ ) )
            {
                return \call_user_func_array( 'parent::' . __FUNCTION__, \func_get_args() );
            }
            else
            {
                throw $e;
            }
        }
    }

 

It only appears to save the default data into $this

 

array (
  'id' => 2,
  'parent_id' => 0,
  'name_seo' => 'test-321',
  'count_imgs' => 0,
  'count_comments' => 0,
  'count_imgs_hidden' => 0,
  'count_comments_hidden' => 0,
  'cover_img_id' => 0,
  'last_img_id' => 0,
  'last_img_date' => 0,
  'sort_options' => 'album_last_img_date',
  'allow_comments' => true,
  'allow_rating' => true,
  'approve_img' => false,
  'approve_com' => false,
  'rating_aggregate' => 0,
  'rating_count' => 0,
  'rating_total' => 0,
  'watermark' => 1,
  'position' => 2,
  'can_tag' => true,
  'preset_tags' => '',
  'public_albums' => 0,
  'nonpublic_albums' => 0,
  'tag_prefixes' => true,
  'allow_albums' => '1',
  'show_rules' => '0',
  'rules_link' => NULL,
  'allow_reviews' => false,
  'review_moderate' => 0,
  'club_id' => NULL,
  'sort_options_img' => 'updated',
  'allow_anonymous' => 0,
  'last_poster_anon' => 0,
)

 

I'll keep trying but hopefully someone can point me in the right direction 🙂

[Daniel F]

Are the data stored to autothreadgen_gallery_addon ? Are you actually loading the data from autothreadgen_gallery_addon and when the object is loaded? 

Without seeing the full code I can only guess..:D

[flashpoint]

17 hours ago, Daniel F said:

Are the data stored to autothreadgen_gallery_addon ? Are you actually loading the data from autothreadgen_gallery_addon and when the object is loaded? 

Without seeing the full code I can only guess..:D

Yes, data is inserted and updated correctly. I believe that the data is loaded but can't seem to find a way to confirm that using \ips\log

 

protected static function constructLoadQuery( $id, $idField, $extraWhereClause )
		{
			try
			{
				return parent::constructLoadQuery( $id, $idField, $extraWhereClause )->join( 'autothreadgen_gallery_addon', 'autothreadgen_gallery_addon.threadgen_gallery_category_id=gallery_categories.category_id' );
				\IPS\Log::log( constructLoadQuery, 'constructLoadQuery' );
			}
			catch ( \RuntimeException $e )
			{
				\IPS\Log::log( $e, 'load query e' );
				if ( method_exists( get_parent_class(), __FUNCTION__ ) )
				{
					\IPS\Log::log( $e, 'load query e' );
					return \call_user_func_array( 'parent::' . __FUNCTION__, \func_get_args() );
				}
				else
				{
					\IPS\Log::log( $e, 'load query e' );
					throw $e;
				}
			}
		}

 

[Daniel F]

You have to override the constructFromData method to set your custom values

[flashpoint]

50 minutes ago, Daniel F said:

You have to override the constructFromData method to set your custom values

Is there any documentation for that? Can't seem to find it on the plugin docs