scaforchio / LAMPSchool

Repository ufficiale di LAMPSchool
GNU Affero General Public License v3.0
20 stars 21 forks source link

[WIP]Credenziali via email #87

Closed MarcoSchiavello closed 1 year ago

MarcoSchiavello commented 3 years ago

reviewer: @valerio-bozzolan

valerio-bozzolan commented 3 years ago

OK @Marco-Schiavello ottimo. Magari definisci una funzione generica chiamata lampschool_url() definita più o meno così:

/**
 * Get a full URL to the LampSchool base directory.
 *
 * It always ends with a slash.
 *
 * Example return value:
 *    http://example.com/dir/
 *
 * @return String
 */
function lampschool_url() {
    $url = null;

    // the user may customize the URL
    if( defined( 'LAMPSCHOOL_URL' ) ) {
        $url = LAMPSCHOOL_URL;
    }

    // as default it's just the current URL
    if( !$url ) {
        $url = $_SERVER[ 'QUALCOSAPROTOCOL' ] . $_SERVER[ 'qualcosahostname'] . $_SERVER['QUALCOSAPATH'];
    }

    return $url;
}

Può essere uno spunto?

Così non tocchiamo l'installatore, troppo lavoro, e sopratutto diamo retro-compatibilità.

Dai che ci siamo!

MarcoSchiavello commented 3 years ago

OK @Marco-Schiavello ottimo. Magari definisci una funzione generica chiamata lampschool_url() definita più o meno così:

/**
 * Get a full URL to the LampSchool base directory.
 *
 * It always ends with a slash.
 *
 * Example return value:
 *    http://example.com/dir/
 *
 * @return String
 */
function lampschool_url() {
    $url = null;

    // the user may customize the URL
    if( defined( 'LAMPSCHOOL_URL' ) ) {
        $url = LAMPSCHOOL_URL;
    }

    // as default it's just the current URL
    if( !$url ) {
        $url = $_SERVER[ 'QUALCOSAPROTOCOL' ] . $_SERVER[ 'qualcosahostname'] . $_SERVER['QUALCOSAPATH'];
    }

    return $url;
}

Può essere uno spunto?

Così non tocchiamo l'installatore, troppo lavoro, e sopratutto diamo retro-compatibilità.

Dai che ci siamo!

ok ho fatto la funzione ho preso il protocollo dall'array $_SERVER mentre il dominio sempre dallo stesso array ma con chiave 'HTTP_HOST' e infine il path per arrivare al registro è la differenza tra il $_SERVER['DOCUMENT_ROOT'] e la root della cartella dove stanno i file. dimmi poi se va bene @valerio-bozzolan

MarcoSchiavello commented 3 years ago

ah ho tolto le costanti che avevo messo e messo gli include_once dove mi avevi detto @valerio-bozzolan

valerio-bozzolan commented 3 years ago

Grande!

Diamo un paio di giorni al buon @vittodevit che si faccia il suo giro in locale e che può dare il suo tocco magico o il suo OK, soprattutto sulla funzione in oggetto.

Dai che ci siamo quasi

vittodevit commented 3 years ago

Grande!

Diamo un paio di giorni al buon @vittodevit che si faccia il suo giro in locale e che può dare il suo tocco magico o il suo OK, soprattutto sulla funzione in oggetto.

Dai che ci siamo quasi

Certo mi pullo la branch di @MarcoSchiavello e ti faccio sapere.

vittodevit commented 1 year ago

Branch in conflitto, probabile integrazione della funzionalità in #108