getPostData() is a magic method, it returns the post data that's already loaded, no extra database calls.
IPSContentCache::update( $post['pid'], 'post', $this->formatPostForCache( $post['post'] ) );
That is the crux of it.... I need to modify the post submission, and I NEED the pid as well, in the same spot, same go, same code exec.
we run that effectively twice then? I'm fairly certain that is heavy... so every mod doing this is stacking another call, when we just need to update the DB and adjust the native data a bit.
While I'm here, add first post has the same issue.... though we have that thankfully via the firstpost flag.... this is a plea for efficiency moreover than anything else..... every other official app has a pre and post for add... why is there an argument even