Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Notice: Undefined offset #349

Open
guillogo opened this issue Nov 18, 2021 · 1 comment
Open

Notice: Undefined offset #349

guillogo opened this issue Nov 18, 2021 · 1 comment

Comments

@guillogo
Copy link

Hi, I'm getting this warning:

Notice: Undefined offset: 2 in /var/www/site/filter/kaltura/filter.php on line 182, triggered 6 times.
Notice: Undefined offset: 3 in /var/www/site/filter/kaltura/filter.php on line 183, triggered 6 times.

Cause: the array count validation is done after trying to get width and height properties here: https://github.com/kaltura/moodle_plugin/blob/MOODLE_311_DEV/filter/kaltura/filter.php#L185
guillogo added a commit to guillogo/moodle_plugin that referenced this issue Nov 18, 2021
@guillogo
Issue kaltura#349: Fix offset in filter.php
d13b8b1 
The array count validation should be done before getting width and height properties.
@guillogo
Copy link
Author

PR created: #350

lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Apr 28, 2022
@lucaboesch
Issue kaltura#349: Fix offset in filter.php
844a353
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Apr 28, 2022
@lucaboesch
Issue kaltura#349: Fix offset in filter.php
80d838b
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue May 14, 2022
@lucaboesch
Issue kaltura#349: Fix offset in filter.php
69d2c34
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Aug 5, 2022
@lucaboesch
Issue kaltura#349: Fix offset in filter.php
3c911e9
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Jan 8, 2023
@lucaboesch
Issue kaltura#349: Fix offset in filter.php
8d06f4e
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Jun 5, 2023
@lucaboesch
Issue kaltura#349: Fix offset in filter.php
8a79dd1
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Jun 5, 2023
@lucaboesch
Issue kaltura#349: Fix offset in filter.php
925e6e6
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Jun 18, 2023
@lucaboesch
Issue kaltura#349: Fix offset in filter.php
1e14627
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Jun 18, 2023
@lucaboesch
Issue kaltura#349: Fix offset in filter.php
23e3fec
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Aug 12, 2023
@lucaboesch
Issue kaltura#349: Fix offset in filter.php
a28fc13
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Feb 5, 2024
@lucaboesch
Fix offset in filter.php resolves kaltura#349
ea094bb 
The array count validation should be done before getting width and height properties.
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Feb 5, 2024
@lucaboesch
Fix offset in filter.php resolves kaltura#349
541bb99 
The array count validation should be done before getting width and height properties.
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue May 26, 2024
@lucaboesch
Fix offset in filter.php resolves kaltura#349
f899bfd 
The array count validation should be done before getting width and height properties.
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Aug 4, 2024
@lucaboesch
Fix offset in filter.php resolves kaltura#349
6315be6 
The array count validation should be done before getting width and height properties.
lucaboesch added a commit to lucaboesch/moodle_plugin that referenced this issue Aug 4, 2024
@lucaboesch
Fix offset in filter.php resolves kaltura#349
3eeeca3 
The array count validation should be done before getting width and height properties.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@guillogo