Zugriff auf Jobliste via API nur mit Login?

30 views
Skip to first unread message

joche...@jobvector.com

unread,
May 24, 2016, 3:43:59 AM5/24/16
to YAWIK
Hallo Carsten,

der Zugriff auf die Jobliste unter /de/api/jobs/organization/organisationID funktioniert nur nach einem Login (in einer älteren Version ging das imho auch ohne Login?) und ist somit nicht mehr von einer externen Seite her aufrufbar. Der ACL-Eintrag 'Jobs/ApiJobList'' in die Jobs/module.config.php:

   .....
           'guest' => array(
                'allow' => array(
                    'Jobboard',
                    'Jobs/Jobboard',
                    'Jobs/ApiJobList',
                    'Jobs/Template' => [ 'view', 'edittemplate' ],
                    'Jobs/Manage' => array(
                        'template',
                    ),
                    'route/lang/jobs/template',
                    'route/lang/jobboard',
                ),
                'deny' => 'JobList'
   ......


ändert daran nichts...

Wo kann ich den Zugriff den für 'guest' freischalten?

VG,
Jochen

joche...@jobvector.com

unread,
May 24, 2016, 3:45:54 AM5/24/16
to YAWIK
Hab's schon gefunden: Der Eintrag muss 'Jobs/ApiJobListByOrganization' heissen.

VG,
Jochen

Carsten Bleek

unread,
May 24, 2016, 4:39:13 AM5/24/16
to ya...@googlegroups.com
Besten Dank für die Info. Das muss natürlich noch in das Release rein.

https://github.com/cross-solution/YAWIK/issues/216

Viele Grüße,

Carsten

joche...@jobvector.com

unread,
May 25, 2016, 11:30:11 AM5/25/16
to YAWIK
Hallo Carsten,

anscheinend werden auch nicht aktive Stellen über die API mit ausgegeben, im Test bei mir kommt eine mit Status 'Entwurf' mit...

VG,
Jochen

joche...@jobvector.com

unread,
May 30, 2016, 6:47:41 AM5/30/16
to YAWIK
Hallo Carsten,

mit der Zeile

'isDraft' => false

in module/Jobs/src/Jobs/Repository/Job.php:

public function findByOrganization($organizationId)
    {
        return $this->findBy([
            'organization' => new \MongoId($organizationId),
            'isDraft' => false
        ]);
    }

scheint das Problem behoben zu sein. Oder siehst du Nebeneffekte?

Vg,
Jochen

Am Mittwoch, 25. Mai 2016 17:30:11 UTC+2 schrieb joche...@jobvector.com:
Hallo Carsten,

Carsten Bleek

unread,
May 30, 2016, 6:54:59 AM5/30/16
to ya...@googlegroups.com
Hallo Jochen,

das ist vollkommen richtig.

Schick uns doch einen Pull Request.

Viele Grüße,

Carsten
> --
> Demo: http://yawik.org/demo
> ---
> Sie erhalten diese Nachricht, weil Sie in Google Groups E-Mails von der
> Gruppe "YAWIK" abonniert haben.
> Wenn Sie sich von dieser Gruppe abmelden und keine E-Mails mehr von
> dieser Gruppe erhalten möchten, senden Sie eine E-Mail an
> yawik+un...@googlegroups.com
> <mailto:yawik+un...@googlegroups.com>.
> Wenn Sie in dieser Gruppe einen Beitrag posten möchten, senden Sie eine
> E-Mail an ya...@googlegroups.com <mailto:ya...@googlegroups.com>.
> Wenn Sie diese Diskussion im Web verfolgen möchten, rufen Sie
> https://groups.google.com/d/msgid/yawik/c619f2ce-450a-42ae-bfb6-605a0d32edd0%40googlegroups.com
> <https://groups.google.com/d/msgid/yawik/c619f2ce-450a-42ae-bfb6-605a0d32edd0%40googlegroups.com?utm_medium=email&utm_source=footer>
> auf.
> Weitere Optionen finden Sie unter https://groups.google.com/d/optout.

joche...@jobvector.com

unread,
May 30, 2016, 9:20:15 AM5/30/16
to YAWIK
Wie mache ich das?

Carsten Bleek

unread,
May 30, 2016, 9:40:35 AM5/30/16
to ya...@googlegroups.com
Hallo Jochen,

das ist ganz einfach. Man forked ein Repository. Dann checked man seinen Fork aus, macht die Änderungen und pushed sie in seinen Fork.

Und dann dückt man in seinem Github Repository auf "Send Pull Request".

https://help.github.com/articles/fork-a-repo/

Ich habe aber die Änderung nochmal besprochen.

Eigentlich müsste man die 3 Methoden

findBy
findByOne
createQueryBuilder

https://github.com/cross-solution/YAWIK/blob/develop/module/Applications/src/Applications/Repository/Application.php#L30

in Jobs/Jobposity/Job kopieren. Dann würde das mit dem isDraft funktionieren.

Weil man Code eigentlich nicht kopieren soll, soll man für diese Logik einen Trait machen.

Am besten machen wir also dafür ein Github Issue. Dann kann man sich in seinem Commit auf den Issue beziehen.

Viele Grüße,

Carsten



Am 30.05.16 um 15:20 schrieb joche...@jobvector.com:
--
Demo: http://yawik.org/demo
---
Sie erhalten diese Nachricht, weil Sie in Google Groups E-Mails von der Gruppe "YAWIK" abonniert haben.
Wenn Sie sich von dieser Gruppe abmelden und keine E-Mails mehr von dieser Gruppe erhalten möchten, senden Sie eine E-Mail an yawik+un...@googlegroups.com.
Wenn Sie in dieser Gruppe einen Beitrag posten möchten, senden Sie eine E-Mail an ya...@googlegroups.com.
Wenn Sie diese Diskussion im Web verfolgen möchten, rufen Sie https://groups.google.com/d/msgid/yawik/35939e57-e0da-4696-9897-1e0b227f7b2e%40googlegroups.com auf.
Reply all
Reply to author
Forward
0 new messages