Conversation
…ct model to be performed on here
@@ -1,6 +1,7 @@ | |||
{ | |||
"name": "atk4/atk4", | |||
"name": "xepan/atk4", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this change is probably a mistake.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep ... :) sure it is ... there are a lot of things i didn't wanted to push
@@ -349,7 +349,7 @@ function verifyCredentials($user, $password) | |||
// Attempt to load user data by username. If not found, return | |||
// false | |||
|
|||
$data = $this->model->newInstance()->tryLoadBy($this->login_field, $user); | |||
$data = $this->model->tryLoadBy($this->login_field, $user); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
riskier than original, why exactly the change?
@@ -132,7 +132,7 @@ function parseRequestedURL(){ | |||
if($https)$url.='s'; | |||
|
|||
// 2. Continue building. We are adding hostname next and port. | |||
$url .= "://".$_SERVER["SERVER_NAME"]; | |||
$url .= "://".$_SERVER["HTTP_HOST"]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will this still work with apache?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was only change i wanted to push btw 😆
$template=$this->template->cloneRegion('field_input'); | ||
$options['show_input_only']=true; | ||
if ($insert_into->template->hasTag($name)) { | ||
if($this->app->getConfig('form/layout/inputonly',true)){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems like some hard-code thing.
// 'addons'=>'addons', | ||
// 'template'=>$templates_folder, | ||
// ))->setBasePath(dirname($base_directory).'/shared'); | ||
// } | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also, probably to big of a change for legacy projects.
Nginx Does not throw dynamic subdomain, $app->url()->absolute()->getBaseURL() will show main server_name set in nginx configuration only. HTTP_HOST is better option.