Skip to content
New issue

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

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

Already on GitHub? # to your account

"No protocol method DateTimeProtocol plus- defined for type null:" error on opening a sent image in chat #15389

Closed
pavloburykh opened this issue Mar 16, 2023 · 14 comments · Fixed by #15390 or #16471
Assignees
Labels

Comments

@pavloburykh
Copy link
Contributor

@OmarBasem probably a regression of some of recent PRs related to images

Steps:

  1. Open any chat
  2. Send a message containing image
  3. Try to open image after sending

Expected result: image opens, no error

Actual result: "No protocol method DateTimeProtocol plus- defined for type null:" error appears, image is not opened. In case with messages containing more than 1 image - it is able to open them after re-entering chat, when images start showing as an album. See video below.

Status-debug-logs (23).zip

telegram-cloud-document-2-5231064321630418851.mp4

Additional Information

  • Status version: nightly
  • Operating System: Android, iOS
@pavloburykh pavloburykh added this to the 1.21.0-rc.2 milestone Mar 16, 2023
@OmarBasem
Copy link
Contributor

Thanks @pavloburykh. There was a small typo. Fixed in #15390

@OmarBasem OmarBasem self-assigned this Mar 16, 2023
@pavloburykh
Copy link
Contributor Author

pavloburykh commented Mar 16, 2023

#15390

Thanx @OmarBasem !

I am facing another issue on IOS and I wonder if it could be somehow related to current one or it is a separate issue? Could you please check?

"Index argument nth must be a number" error appears on opening album image on IOS

There are no clear steps. I have managed to reproduce the bug couple of times using these one:

  1. Open chat
  2. Send message containing 6 images
  3. Re open chat
  4. Send message containing 1 image
  5. Open the the last message to receive "No protocol method DateTimeProtocol plus- defined for type null:" error
  6. Try to open different images in the previously sent message containing album

Actual result: "Index argument nth must be a number" error appears

status_logs (1).zip

telegram-cloud-document-2-5231064321630418883.mp4

@OmarBasem
Copy link
Contributor

@pavloburykh no it's not related. This happens randomly. And I believe it can happen on Android also.

@pavloburykh
Copy link
Contributor Author

@OmarBasem thanx. I will log this issue separately. Will try to figure out more clear steps and try reproduce it on Android. By now I have faced it only on IOS.

@OmarBasem
Copy link
Contributor

OmarBasem commented Mar 16, 2023

Okay thanks @pavloburykh. Would be good if you can find a clearer way of reproducing it, and whether it is on both platforms.

@pavloburykh
Copy link
Contributor Author

@OmarBasem hi! Re-opening the bug as it is easily reproducible on IOS.

Steps:

  1. Open any chat
  2. Send 1 image
  3. Open and close the image right after it is sent
  4. Observe an error

Actual result: No protocol method DateTimeProtocol plus- defined for type null:" error on opening a sent image in chat.

telegram-cloud-document-2-5370594778146679901.mp4

@churik
Copy link
Member

churik commented Jun 1, 2023

Can't easy reproduce it anymore

@churik churik closed this as completed Jun 1, 2023
@pavloburykh
Copy link
Contributor Author

Can't easy reproduce it anymore

I am easily reproducing the issue on my iPhone X, IOS 14.1 on latest nightly build. Let's leave this issue opened.

telegram-cloud-document-2-5461147317404970244.mp4

@flexsurfer
Copy link
Member

hey @OmarBasem do you plan to work on it?

@OmarBasem
Copy link
Contributor

@flexsurfer Yes I will look into it

@OmarBasem
Copy link
Contributor

@pavloburykh are you still reproducing this issue? I am not able to reproduce it

@pavloburykh
Copy link
Contributor Author

pavloburykh commented Jun 27, 2023

@pavloburykh are you still reproducing this issue? I am not able to reproduce it

@OmarBasem still reproducing (latest nightly, IOS 14.1, iPhone X) but with another error

photo_2023-06-27 12 31 51

telegram-cloud-document-2-5249055527316500493.mp4

@OmarBasem
Copy link
Contributor

OmarBasem commented Jun 27, 2023

@pavloburykh are you getting this error before or after that PR which was merged very recently: #16405.

@pavloburykh
Copy link
Contributor Author

#16405

@OmarBasem I am getting it in today's nightly which does not include this fix #16405

I have just checked build from this PR #16405 and still getting error there.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants