Repository navigation
Cascata (Moodle 4.1): #11 e #12 - #13
Merged
Merged
Conversation
Numa branch de trabalho (fix-*-405), o job versions caía em
DEFAULT_VERSIONS ("30 38") e testava versões que o plugin não suporta
ali: falso vermelho com pluginrequirementsnotmet. O PR da branch testa
a versão da base. Custo aceito: push em branch sem PR fica sem CI.
Closes #10
…_string() - privacy/provider: get_reason() sem ": string" era erro fatal ao carregar a classe no Moodle 3.8 e no 4.5. Passa a usar core_privacy legacy_polyfill (_get_reason()), que continua válido no PHP 5.6 do Moodle 3.0. - restore: as mensagens "failed to rewrite" e "truncated at" saem de get_string() (errorrewritefailed, logtruncated), com tradução em pt_br. O texto em inglês no log não muda. - Testes: provider carregável; mensagens vindas do pacote de idioma, com a fronteira do limite de 8192 bytes. - Versão 2026092901.
Da revisão do #11: tag não dispara mais (só `branches` definido), e o DEFAULT_VERSIONS vale para toda branch que o job versions não conhece, inclusive disparo manual e PR contra branch de trabalho. Uma MOODLE_XX_STABLE nova precisa da sua linha no job.
- provider: phpcs:disable PSR2.Methods.MethodDeclaration.Underscore em volta de _get_reason(), nome imposto pelo legacy_polyfill do core. - fixtures dos testes de log: uma classe por arquivo. - privacy_provider_test: sem MOODLE_INTERNAL (arquivo sem efeito colateral). - log_messages_test: compara com o texto literal que o log tinha, não com o próprio get_string(); o tearDown restaura o string manager anterior; teste novo confere que en e pt_br têm as mesmas chaves e placeholders; docblock deixa de afirmar que todo texto do log vem do pacote de idioma (o cabeçalho dos pedaços de conteúdo continua literal).
CI: push só na main e nas MOODLE_*_STABLE (#10)
Fix: provider de privacidade compatível com a interface; logs via get_string()
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cascata dos #11 (gatilho
pushdo CI) e #12 (provider de privacidade e logs viaget_string()) para aMOODLE_401_STABLE.main, incluindo olegacy_polyfillno provider, igual em todas as branches.log_messages_test,privacy_provider_test) com os ajustes do PHPUnit 9:: voidnos testes,setUp(): void/tearDown(): voidcomparent::,: arrayno data provider.version.php:2026092901, mantendo orequiresdo 4.1. Era o único conflito.php -lsem erros no PHP 8.3. O PHP 8.0 e os testes no 4.1 ficam por conta deste CI.Ordem de merge: este PR →
MOODLE_401_STABLE→ PR daMOODLE_405_STABLE.🤖 Generated with Claude Code