Add Issue numbers to @todo comments

figuren-theater · Oct 16, 2023 · 70cdefa · 70cdefa
1 parent 6323658
commit 70cdefa
Show file tree

Hide file tree

Showing 7 changed files with 19 additions and 17 deletions.
diff --git a/ft-network-sourcelinks.php b/ft-network-sourcelinks.php
@@ -84,7 +84,7 @@ public static function get_subscribed_events() : Array
 
 
 
-			// TEMP DISABLED, until sync works, read on down the code
+			// @todo #32 Re-ENABLE - TEMP DISABLED; until sync works, read on down the code
 			// 'after_setup_theme' => 'enable__on_setup_theme', // working
 
 
@@ -204,12 +204,13 @@ public function i18n()
 
 
 	/**
-	 * @todo TEMPORARILY DISABLED
+	 * @todo #32
+	 *       TEMPORARILY DISABLED
 	 *       but needed when sync/importing starts
-	 *       so we can assign different post_formats based on the 
+	 *       so we can assign different post_formats based on the
 	 *       source_links post_format
 	 *
-	 * 
+	 *
 	 * Note that you must call 'add_theme_support()' before the init hook gets called!
 	 *
 	 * A good hook to use is the after_setup_theme hook.

diff --git a/inc/Network/Options/Preset__wpseo_social.php b/inc/Network/Options/Preset__wpseo_social.php
@@ -94,8 +94,8 @@ function( $v, $k ) use ( &$_urls, $_social_links )   {
 				// clean up url as pseudo label
 				// so this makes
 				// https://instagram.com -> instagram
-			
-				// @TODO
+
+				// @todo #25
 				// we have this string-cleaning now 3 times
 				// 
 				// - plugins\ft-network-sourcelinks\src\block-editor\blocks\filtered-links\index.php#L90
@@ -179,7 +179,7 @@ public function pre_option_wpseo_social( array|bool $wpseo_social ) : array {
 
 		if ( isset( $_urls['pinterest'] ) && ( !isset( $wpseo_social['pinterest_url'] ) || empty( $wpseo_social['pinterest_url'] ))) {
 			$wpseo_social['pinterest_url'] = $_urls['pinterest'];
-			// @TODO add handling for post_meta 'pinterestverify'
+			// @TODO #26 add handling for post_meta 'pinterestverify'
 		}
 
 		if ( isset( $_urls['twitter'] ) && ( !isset( $wpseo_social['twitter_site'] ) || empty( $wpseo_social['twitter_site'] ))) {

diff --git a/inc/Network/Post_Types/Post_Type__ft_link.php b/inc/Network/Post_Types/Post_Type__ft_link.php
@@ -327,8 +327,8 @@ public static function get_readable_link( WP_Post|null $post = null ) {
 		if( null === $post )
 			global $post;
 
-		
-		// @TODO
+
+		// @todo #25
 		// we have this string-cleaning now 3 times
 		// 
 		// - plugins\ft-network-sourcelinks\src\block-editor\blocks\filtered-links\index.php#L90
@@ -421,8 +421,8 @@ public static function get_instance()
 
 
 	/**
-	 * @todo Everything.
-	 * 
+	 * @todo #27 Refactor __is_ft_link_privacy_relevant()
+	 *
 	 * [__is_ft_link_privacy_relevant description]
 	 *
 	 * @package [package]

diff --git a/inc/Network/Taxonomies/Taxonomy__ft_link_shadow.php b/inc/Network/Taxonomies/Taxonomy__ft_link_shadow.php
@@ -105,7 +105,7 @@ protected function prepare_labels() : array
 			# Override the base names used for labels:
 			'singular' => __('Import Source','figurentheater'),
 			'plural'   => __('Import Sources','figurentheater'),
-			'slug'     => '' #TODO
+			'slug'     => '' #TODO #28 Add documenatation on, why this is '' empty here ?!
 		];
 	}
 

diff --git a/src/block-editor/blocks/filtered-links/index.php b/src/block-editor/blocks/filtered-links/index.php
@@ -59,7 +59,8 @@ function render_block( $attributes, $content, $block ) {
 		return '';
 
 
-	// TODO // not used at the moment, so it defaults to: div
+	// TODO #29 // consider using $attributes['tagName']
+	// which is not used at the moment, so it defaults to: ul
 	$tag_name = empty( $attributes['tagName'] ) ? 'ul' : $attributes['tagName'];
 
 	// get and merge wrapper attributes with text-align CSS class
@@ -78,8 +79,8 @@ function render_block( $attributes, $content, $block ) {
 		function( $link ) use ( $attributes )
 		{
 			// die(var_export([$_url,$link->post_content],true));
-			
-			// @TODO
+
+			// @todo #25
 			// we have this string-cleaning now 3 times
 			// 
 			// - plugins\ft-network-sourcelinks\src\block-editor\blocks\filtered-links\index.php#L90

diff --git a/src/block-editor/blocks/simple-links/edit.js.BU b/src/block-editor/blocks/simple-links/edit.js.BU
@@ -209,7 +209,7 @@ function useLinkData() {
 	);
 
 	return useMemo( () => {
-		/*// TODO: Once the REST API supports passing multiple values to
+		/*// TODO: #30 Once the REST API supports passing multiple values to
 		// 'orderby', this can be removed.
 		// https://core.trac.wordpress.org/ticket/39037
 		const sortedPages = sortBy( pages, [ 'menu_order', 'title.rendered' ] );

diff --git a/src/block-editor/blocks/social-links/edit.js b/src/block-editor/blocks/social-links/edit.js
@@ -136,7 +136,7 @@ export default function Edit() {
         []
     );
 
-    // @todo
+    // @todo #31
     // check for more than one InnerBlocks
     // and only then
     // look for the links