From 25ecff16e7f7972a5a7b12f7e563af8c9a27fd7a Mon Sep 17 00:00:00 2001
From: ruru4143 <ruru@event-federation.eu>
Date: Mon, 11 Dec 2023 19:43:50 +0100
Subject: [PATCH] adding todo's

---
 includes/transformer/class-base.php | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/includes/transformer/class-base.php b/includes/transformer/class-base.php
index 248fb6d..5a9ad96 100644
--- a/includes/transformer/class-base.php
+++ b/includes/transformer/class-base.php
@@ -25,7 +25,7 @@ use function Activitypub\site_supports_blocks;
  *
  *
  */
-abstract class Base {
+abstract class Base { // todo renmae to Base_tranformer
 	/**
 	 * The WP_Post object.
 	 *
@@ -341,6 +341,8 @@ abstract class Base {
 	 * @return string The User-URL.
 	 */
 	protected function get_attributed_to() {
+		// todo is this descriptive enough? what if for some people the event should be attributed to the venue?
+		// i would maybe prefer something like "get post author??"
 		if ( is_single_user() ) { // todo what is if this setting gets changed
 			$user = new Blog_User();
 			return $user->get_url();
@@ -691,7 +693,7 @@ abstract class Base {
 	 */
 	protected function get_basic_content_map() {
 		return array(
-			$this->get_locale() => $this->get_content(),
+			$this->get_locale() => $this->get_content(), // todo is this a valid assumption?
 		);
 	}
 }