Skip to content

Commit 4391757

Browse files
committed
fix: missing filter at badge scans csv export
Signed-off-by: romanetar <roman_ag@hotmail.com>
1 parent 788e00f commit 4391757

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

app/Http/Controllers/Apis/Protected/Summit/OAuth2SummitBadgeScanApiController.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -510,7 +510,7 @@ function($filter) use($summit, $current_member){
510510
if (!is_null($current_member)){
511511
if ($current_member->isAuthzFor($summit)) return $filter;
512512
// add filter for sponsor user
513-
if ($current_member->isSponsorUser()) {
513+
if ($current_member->isSponsorUser() || $current_member->isExternalSponsorUser()) {
514514
$sponsor_ids = $current_member->getSponsorMembershipIds($summit);
515515
// is allowed sponsors are empty, add dummy value
516516
if (!count($sponsor_ids)) $sponsor_ids[] = 0;

tests/oauth2/OAuth2SummitBadgeScanApiControllerTest.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ public function testAddEncryptedBadgeScan(){
109109
$this->assertEquals($badge->getId(), $badge_scan->badge_id);
110110
}
111111

112-
public function testAddBadgeScanWithOneSponsorsPerMember(){
112+
public function testAddBadgeScanWithOneSponsorPerMember(){
113113
self::$member->clearGroups();
114114
self::$member->add2Group($this->sponsor_group);
115115
self::$em->persist(self::$member);
@@ -391,7 +391,7 @@ public function testAddBadgeScanWithMultipleSponsorsWithSponsorId()
391391
}
392392

393393
public function testUpdateBadgeScan(){
394-
$scan = $this->testAddBadgeScanWithOneSponsorsPerMember();
394+
$scan = $this->testAddBadgeScanWithOneSponsorPerMember();
395395

396396
$params = [
397397
'id' => self::$summit->getId(),
@@ -510,7 +510,7 @@ public function testGetAllSummitBadgeScans(){
510510
}
511511

512512
public function testGetSummitBadgeScan(){
513-
$badge_scan = $this->testAddBadgeScanWithOneSponsorsPerMember();
513+
$badge_scan = $this->testAddBadgeScanWithOneSponsorPerMember();
514514

515515
$params = [
516516
'id' => self::$summit->getId(),
@@ -537,7 +537,7 @@ public function testGetSummitBadgeScan(){
537537

538538
public function testExportSummitBadgeScans(){
539539

540-
$this->testAddBadgeScanWithOneSponsorsPerMember();
540+
$this->testAddBadgeScanWithOneSponsorPerMember();
541541

542542
$params = [
543543
'id' => self::$summit->getId(),
@@ -561,7 +561,7 @@ public function testExportSummitBadgeScans(){
561561

562562
public function testExportSummitBadgeScansWithReportSettingsRestriction(){
563563

564-
$this->testAddBadgeScanWithOneSponsorsPerMember();
564+
$this->testAddBadgeScanWithOneSponsorPerMember();
565565

566566
$sponsor = self::$summit->getSummitSponsors()[0];
567567
if (!$sponsor instanceof Sponsor) self::fail();
@@ -622,7 +622,7 @@ public function testExportSummitBadgeScansWithReportSettingsRestriction(){
622622

623623
public function testExportSummitBadgeScansWithAllReportSettingsRestriction(){
624624

625-
$this->testAddBadgeScanWithOneSponsorsPerMember();
625+
$this->testAddBadgeScanWithOneSponsorPerMember();
626626

627627
$sponsor = self::$summit->getSummitSponsors()[0];
628628
if (!$sponsor instanceof Sponsor) self::fail();

0 commit comments

Comments
 (0)