8374: Addressing review comments

showmore_dyn/showmore_dyn.css

@@ -1,7 +1,7 @@
 .wall-item-body-toggle {
 	width: 100%;
 	text-align: center;
-	background-image: linear-gradient(rgba(0,0,0,0), #f6f6f6);
+	background-image: linear-gradient(rgba(0, 0, 0, 0), #f6f6f6);
 	cursor: pointer;
 	height: 50px;
 	position: absolute;
@@ -11,6 +11,7 @@
 	left: 0;
 	display: none;
 }
+
 .wall-item-body-toggle-text {
 	width: 100%;
 	position: absolute;
@@ -20,22 +21,23 @@
 	left: 0;
 	text-shadow: 0 0 5px #f6f6f6;
 	font-weight: bold;
-	background: none!important;
+	background: none !important;
 	border: none;
-	padding: 0!important;
+	padding: 0 !important;
 	cursor: pointer;
 }
+
 .wall-item-body-toggle-text:hover {
-	text-decoration:underline;
+	text-decoration: underline;
 }
+
 .wall-item-body {
 	position: relative;
 	width: 100%;
 }
+
 .wall-item-body-wrapper {
 	position: relative;
 	width: 100%;
 	display: inline-block;
 }
-
-

showmore_dyn/showmore_dyn.js

@@ -1,6 +1,6 @@
 var nextBodyIdx = 0;
 
-$(document).ready(function(){
+$(document).ready(function() {
 	$("head").append('<style type="text/css"></style>');
 	var newStyleElement = $("head").children(':last');
 	newStyleElement.html('.limit-height{max-height: ' + postLimitHeight + 'px; overflow: hidden; display:inline-block;}');
@@ -28,15 +28,22 @@ function handleNewWallItemBodies() {
 		var limited = processHeightLimit($el);
 
 		if (!limited) {
-			var mutationObserver = new MutationObserver(function(mutations) {
+			var mutationObserver = new MutationObserver(function() {
 				var limited = processHeightLimit($el);
 				if (limited) {
 					mutationObserver.disconnect()
 				}
 			});
-			mutationObserver.observe($el[0], { attributes: true, characterData: true, childList: true, subtree: true, attributeOldValue: true, characterDataOldValue: true });
+			mutationObserver.observe($el[0], {
+				attributes: true,
+				characterData: true,
+				childList: true,
+				subtree: true,
+				attributeOldValue: true,
+				characterDataOldValue: true
+			});
 
-			$el.imagesLoaded().then(function(){
+			$el.imagesLoaded().then(function() {
 				processHeightLimit($el);
 			});
 		}
@@ -79,5 +86,3 @@ function processHeightLimit($item) {
 		return true;
 	}
 }
-
-

showmore_dyn/showmore_dyn.php

@@ -15,21 +15,13 @@ use Friendica\Core\Renderer;
 use Friendica\Database\DBA;
 use Friendica\DI;
 
-function showmore_dyn_install() {
+function showmore_dyn_install()
-	Hook::register('page_end', 'addon/showmore_dyn.php/showmore_dyn.php', 'showmore_dyn_script');
-	Hook::register('head'  , __FILE__, 'showmore_dyn_head');
-	Hook::register('footer', __FILE__, 'showmore_dyn_footer');
-	Hook::register('addon_settings', 'addon/showmore_dyn/showmore_dyn.php', 'showmore_dyn_settings');
-        Hook::register('addon_settings_post', 'addon/showmore_dyn/showmore_dyn.php', 'showmore_dyn_settings_post');
-}
-
-function showmore_dyn_uninstall()
 {
-	Hook::unregister('page_end', 'addon/jappixmini/jappixmini.php', 'jappixmini_script');
+	Hook::register('page_end',  __FILE__, 'showmore_dyn_script');
-	Hook::unregister('head'  , __FILE__, 'showmore_dyn_head');
+	Hook::register('head', __FILE__, 'showmore_dyn_head');
-	Hook::unregister('footer', __FILE__, 'showmore_dyn_footer');
+	Hook::register('footer', __FILE__, 'showmore_dyn_footer');
-        Hook::unregister('addon_settings', 'addon/showmore_dyn/showmore_dyn.php', 'showmore_dyn_settings');
+	Hook::register('addon_settings',  __FILE__, 'showmore_dyn_settings');
-        Hook::unregister('addon_settings_post', 'addon/showmore_dyn/showmore_dyn.php', 'showmore_dyn_settings_post');
+	Hook::register('addon_settings_post',  __FILE__, 'showmore_dyn_settings_post');
 }
 
 function showmore_dyn_head(App $a, &$b)
@@ -42,35 +34,47 @@ function showmore_dyn_footer(App $a, &$b)
 	DI::page()->registerFooterScript(__DIR__ . '/showmore_dyn.js');
 }
 
-function showmore_dyn_settings_post(){
+function showmore_dyn_settings_post()
-        if(! local_user())
+{
+	if(!local_user()) {
 		return;
-	if (isset($_POST['showmore_dyn-submit'])){
+	}
+
+	if (isset($_POST['showmore_dyn-submit'])) {
 		$limitHeight = $_POST['showmore_dyn_height'];
+		if ($limitHeight && is_numeric($limitHeight)) {
 			DI::pConfig()->set(local_user(), 'showmore_dyn', 'limitHeight', $limitHeight);
 		}
+	}
 }
 
-function showmore_dyn_settings(&$a,&$o) {
+function showmore_dyn_settings(&$a, &$o)
-    if(! local_user())
+{
+	if(!local_user()) {
 		return;
+	}
 
+	$limitHeight = DI::pConfig()->get(local_user(), 'showmore_dyn', 'limitHeight', 250);
+	DI::pConfig()->set(local_user(), 'showmore_dyn', 'limitHeight', $limitHeight);
 
-        $limitHeight = DI::pConfig()->get(local_user(), 'showmore_dyn', 'limitHeight' );
+	$t = Renderer::getMarkupTemplate('settings.tpl', 'addon/showmore_dyn/');
-        if ($limitHeight=='') { $limitHeight = 250; DI::pConfig()->set(local_user(), 'showmore_dyn', 'limitHeight', $limitHeight); }
-
-        $t = Renderer::getMarkupTemplate("settings.tpl", "addon/showmore_dyn/");
 	$o .= Renderer::replaceMacros($t, [
 		'$submit' => DI::l10n()->t('Save Settings'),
-                '$title' => "Showmore Dynamic",
+		'$title' => 'Showmore Dynamic',
 		'$label' => DI::l10n()->t('Limit Height'),
 		'$limitHeight' => $limitHeight,
 	]);
 
 }
 
-function showmore_dyn_script() {
+function showmore_dyn_script()
-	$limitHeight = DI::pConfig()->get(local_user(), 'showmore_dyn', 'limitHeight' );
+{
+	$limitHeight = DI::pConfig()->get(local_user(), 'showmore_dyn', 'limitHeight', 250);
 	$showmore_dyn_showmore_linktext = DI::l10n()->t('Show more ...');
-	DI::page()['htmlhead'] .= '<script>var postLimitHeight = ' . intval($limitHeight) . '; var showmore_dyn_showmore_linktext = "' . $showmore_dyn_showmore_linktext . '"</script>';
+	DI::page()['htmlhead'] .= <<<EOT
+<script>
+	var postLimitHeight = $limitHeight;
+	var showmore_dyn_showmore_linktext = "$showmore_dyn_showmore_linktext";
+</script>
+EOT;
 }

showmore_dyn/templates/settings.tpl

@@ -9,4 +9,3 @@
 		<input type="submit" value="{{$submit}}" class="settings-submit" name="showmore_dyn-submit" />
 	</div>
 </div>
-