Commit 677912db945ddda050218ca398ad63ecd7bef4f4
1 parent
897ce8af
Exists in
master
Alterado metodo getEscolaSerieDisciplina da classe App_Model_IedFinder para lanç…
…ar erro, caso um dos parametros serieId ou escolaId seja nulo, corrigindo assim um que ocorre no cadastro de nova turma, onde não é enviado código da série nem escola o que faz com que este metodo carregue todas as disciplinas, alocando mais memória do que o permitido no arquivo /.htaccess (64MB)
Showing
1 changed file
with
18 additions
and
12 deletions
Show diff stats
ieducar/lib/App/Model/IedFinder.php
@@ -335,8 +335,8 @@ class App_Model_IedFinder extends CoreExt_Entity | @@ -335,8 +335,8 @@ class App_Model_IedFinder extends CoreExt_Entity | ||
335 | * Retorna array com as referências de pmieducar.escola_serie_disciplina | 335 | * Retorna array com as referências de pmieducar.escola_serie_disciplina |
336 | * a modules.componente_curricular ('ref_ref_cod_disciplina'). | 336 | * a modules.componente_curricular ('ref_ref_cod_disciplina'). |
337 | * | 337 | * |
338 | - * @param int $anoEscolar O código do ano escolar/série. | ||
339 | - * @param int $escola O código da escola. | 338 | + * @param int $serieId O código do ano escolar/série. |
339 | + * @param int $escola O código da escola. | ||
340 | * @param ComponenteCurricular_Model_ComponenteDataMapper $mapper (Opcional) | 340 | * @param ComponenteCurricular_Model_ComponenteDataMapper $mapper (Opcional) |
341 | * Instância do mapper para recuperar todas as instâncias persistidas de | 341 | * Instância do mapper para recuperar todas as instâncias persistidas de |
342 | * ComponenteCurricular_Model_Componente atribuídas no ano escolar/série da | 342 | * ComponenteCurricular_Model_Componente atribuídas no ano escolar/série da |
@@ -345,20 +345,26 @@ class App_Model_IedFinder extends CoreExt_Entity | @@ -345,20 +345,26 @@ class App_Model_IedFinder extends CoreExt_Entity | ||
345 | * @throws App_Model_Exception caso não existam componentes curriculares | 345 | * @throws App_Model_Exception caso não existam componentes curriculares |
346 | * atribuídos ao ano escolar/série da escola. | 346 | * atribuídos ao ano escolar/série da escola. |
347 | */ | 347 | */ |
348 | - public static function getEscolaSerieDisciplina($anoEscolar, $escola, | 348 | + public static function getEscolaSerieDisciplina($serieId, $escolaId, |
349 | ComponenteCurricular_Model_ComponenteDataMapper $mapper = NULL, | 349 | ComponenteCurricular_Model_ComponenteDataMapper $mapper = NULL, |
350 | $disciplinaId = null) | 350 | $disciplinaId = null) |
351 | { | 351 | { |
352 | + if (is_null($serieId)) | ||
353 | + throw new App_Model_Exception('O parametro serieId não pode ser nulo'); | ||
354 | + | ||
355 | + if (is_null($escolaId)) | ||
356 | + throw new App_Model_Exception('O parametro escolaId não pode ser nulo'); | ||
357 | + | ||
352 | // Disciplinas na série na escola | 358 | // Disciplinas na série na escola |
353 | $escolaSerieDisciplina = self::addClassToStorage('clsPmieducarEscolaSerieDisciplina', | 359 | $escolaSerieDisciplina = self::addClassToStorage('clsPmieducarEscolaSerieDisciplina', |
354 | NULL, 'include/pmieducar/clsPmieducarEscolaSerieDisciplina.inc.php'); | 360 | NULL, 'include/pmieducar/clsPmieducarEscolaSerieDisciplina.inc.php'); |
355 | 361 | ||
356 | - $disciplinas = $escolaSerieDisciplina->lista($anoEscolar, $escola, $disciplinaId, 1); | 362 | + $disciplinas = $escolaSerieDisciplina->lista($serieId, $escolaId, $disciplinaId, 1); |
357 | 363 | ||
358 | if (FALSE === $disciplinas) { | 364 | if (FALSE === $disciplinas) { |
359 | throw new App_Model_Exception(sprintf( | 365 | throw new App_Model_Exception(sprintf( |
360 | 'Nenhuma disciplina para a série (%d) e a escola (%d) informados', | 366 | 'Nenhuma disciplina para a série (%d) e a escola (%d) informados', |
361 | - $anoEscolar, $escola | 367 | + $serieId, $escolaId |
362 | )); | 368 | )); |
363 | } | 369 | } |
364 | 370 | ||
@@ -372,15 +378,15 @@ class App_Model_IedFinder extends CoreExt_Entity | @@ -372,15 +378,15 @@ class App_Model_IedFinder extends CoreExt_Entity | ||
372 | $componentes[] = $componente; | 378 | $componentes[] = $componente; |
373 | } | 379 | } |
374 | 380 | ||
375 | - return self::_hydrateComponentes($componentes, $anoEscolar, $mapper); | 381 | + return self::_hydrateComponentes($componentes, $serieId, $mapper); |
376 | } | 382 | } |
377 | 383 | ||
378 | /** | 384 | /** |
379 | * Retorna as instâncias de ComponenteCurricular_Model_Componente de uma turma. | 385 | * Retorna as instâncias de ComponenteCurricular_Model_Componente de uma turma. |
380 | * | 386 | * |
381 | - * @param int $anoEscolar O código do ano escolar/série da turma. | ||
382 | - * @param int $escola O código da escola da turma. | ||
383 | - * @param int $turma O código da turma. | 387 | + * @param int $serieId O código do ano escolar/série da turma. |
388 | + * @param int $escola O código da escola da turma. | ||
389 | + * @param int $turma O código da turma. | ||
384 | * @param ComponenteCurricular_Model_TurmaDataMapper $mapper (Opcional) Instância | 390 | * @param ComponenteCurricular_Model_TurmaDataMapper $mapper (Opcional) Instância |
385 | * do mapper para selecionar todas as referências de | 391 | * do mapper para selecionar todas as referências de |
386 | * ComponenteCurricular_Model_Componente persistidas para a turma. | 392 | * ComponenteCurricular_Model_Componente persistidas para a turma. |
@@ -389,7 +395,7 @@ class App_Model_IedFinder extends CoreExt_Entity | @@ -389,7 +395,7 @@ class App_Model_IedFinder extends CoreExt_Entity | ||
389 | * ComponenteCurricular_Model_Componente recuperadas por $mapper. | 395 | * ComponenteCurricular_Model_Componente recuperadas por $mapper. |
390 | * @return array | 396 | * @return array |
391 | */ | 397 | */ |
392 | - public static function getComponentesTurma($anoEscolar, $escola, $turma, | 398 | + public static function getComponentesTurma($serieId, $escola, $turma, |
393 | ComponenteCurricular_Model_TurmaDataMapper $mapper = NULL, | 399 | ComponenteCurricular_Model_TurmaDataMapper $mapper = NULL, |
394 | ComponenteCurricular_Model_ComponenteDataMapper $componenteMapper = NULL, | 400 | ComponenteCurricular_Model_ComponenteDataMapper $componenteMapper = NULL, |
395 | $componenteCurricularId = null) | 401 | $componenteCurricularId = null) |
@@ -408,7 +414,7 @@ class App_Model_IedFinder extends CoreExt_Entity | @@ -408,7 +414,7 @@ class App_Model_IedFinder extends CoreExt_Entity | ||
408 | 414 | ||
409 | // Não existem componentes específicos para a turma | 415 | // Não existem componentes específicos para a turma |
410 | if (0 == count($componentesTurma)) { | 416 | if (0 == count($componentesTurma)) { |
411 | - return self::getEscolaSerieDisciplina($anoEscolar, $escola, $componenteMapper, $componenteCurricularId); | 417 | + return self::getEscolaSerieDisciplina($serieId, $escola, $componenteMapper, $componenteCurricularId); |
412 | } | 418 | } |
413 | 419 | ||
414 | $componentes = array(); | 420 | $componentes = array(); |
@@ -421,7 +427,7 @@ class App_Model_IedFinder extends CoreExt_Entity | @@ -421,7 +427,7 @@ class App_Model_IedFinder extends CoreExt_Entity | ||
421 | $componentes[] = $componente; | 427 | $componentes[] = $componente; |
422 | } | 428 | } |
423 | 429 | ||
424 | - return self::_hydrateComponentes($componentes, $anoEscolar, $componenteMapper); | 430 | + return self::_hydrateComponentes($componentes, $serieId, $componenteMapper); |
425 | } | 431 | } |
426 | 432 | ||
427 | /** | 433 | /** |