Skip to content
This repository

Fix for callio.pe installer not working if callio.pe is in a subdirectory #2

Closed
wants to merge 1 commit into from

2 participants

Andrew Nagy Moshe Brevda
Andrew Nagy

-css/js/imgs would not include based on assumptions that the base would always be 'modules/framework' when it could very well be '< anything >/modules/framework/'

Fix for callio.pe installer not working if callio.pe is in a subdirec…
…tory

-css/js/imgs would not include based on assumptions that the base would always be 'modules/framework' when it could very well be '<anything>/modules/framework/'
f2aafdf
Andrew Nagy

Well it's dynamically generated in this patch. However thats only for the installer setup.php view.

I think it should be in $amp_conf, then the global is available to any and all modules, etc...

I can update the commit and add that if you think it'd be more useful.

Moshe Brevda
Owner

I believe the path issue has been addressed by 8a02b4c and 03e3ffc

Moshe Brevda mbrevda closed this April 22, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Apr 20, 2012
Fix for callio.pe installer not working if callio.pe is in a subdirec…
…tory

-css/js/imgs would not include based on assumptions that the base would always be 'modules/framework' when it could very well be '<anything>/modules/framework/'
f2aafdf
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 1 addition and 1 deletion. Show diff stats Hide diff stats

  1. 2  modules/framework/views/header.php
2  modules/framework/views/header.php
@@ -21,7 +21,7 @@
21 21
 		. '<meta http-equiv="X-UA-Compatible" content="chrome=1">'
22 22
 		. '<meta name="robots" content="noindex" />'
23 23
 		. '<meta name="robots" content="noindex" />'
24  
-		. '<base href="/modules/framework/" />'//TODO: base should not point to framework!!
  24
+		. '<base href="'.dirname($_SERVER['PHP_SELF']).'/modules/framework/" />'//TODO: base should not point to framework!!
25 25
 		. '<link rel="shortcut icon" href="images/favicon.ico">';
26 26
 //css
27 27
 $mainstyle_css      = $amp_conf['BRAND_CSS_ALT_MAINSTYLE'] 
Commit_comment_tip

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.