V3 beta: Organisational logo not on login page

--- login_library.php.orig      2015-07-17 21:25:07.000000000 +0100
+++ login_library.php   2015-07-22 14:00:52.307272855 +0100
@@ -222,7 +222,7 @@
     {
         ?>
         <div
-            style="width:50%; height:100%; float:right; position:relative; background-image:url(<?php echo $xerte_toolkits_site->root_file_path . "branding/logo_right.png";?>); background-repeat:no-repeat; background-position:right; margin-right:10px; float:right">
+            style="width:50%; height:100%; float:right; position:relative; background-image:url(branding/logo_right.png); background-repeat:no-repeat; background-position:right; margin-right:10px; float:right">
         </div>
     <?php
     }
@@ -236,7 +236,7 @@
     if (file_exists($xerte_toolkits_site->root_file_path . "branding/logo_left.png"))
     {
         ?>
-        <img src="<?php echo $xerte_toolkits_site->root_file_path . "branding/logo_left.png";?>" style="margin-left:10px; float:left"/>
+        <img src="branding/logo_left.png" style="margin-left:10px; float:left"/>
     <?php
     }
     else {

Supporting each other

jayaich Topic Author Offline Premium Member Posts: 92 Thank you received: 5	Hmm, seems the patch wasn't attached. Second time round...
	Please Log in or Create an account to join the conversation.

jayaich Topic Author Offline Premium Member Posts: 92 Thank you received: 5	Ron Mitchell wrote: Have you also subscribed to our dev list? lists.nottingham.ac.uk/mailman/listinfo/xerte-dev We're always keen for additional developers to get involved. Well thanks for that, but I'm not really a developer as such but an aged sysadmin :-) However, if I find a bug, then I usually try and sort it out if possible (and pass the fix on). John.
	Please Log in or Create an account to join the conversation.

jayaich Topic Author Offline Premium Member Posts: 92 Thank you received: 5	There were two parts to this bug and fix - one for the 'logo_right' and one for 'logo_left'. It seems that only the 'logo_left' has been fixed in github.
	Please Log in or Create an account to join the conversation.