[Pull] Refactor commits #452

Closed
opened 2 years ago by mmarif · 5 comments
mmarif commented 2 years ago
Owner

Closes #440

Right now it will crash for whatever reason of using ?attr/ which is really annoying.

I had this issue last time, can't remember how I fixed it.

If you remove the custom attr from list_commits.xml, it will work. Atleast it's the case on my side.

Edit: fixed the crash.


Original Pull att :https://gitea.com/gitnex/GitNex/pulls/452

Merged with: 195bfc8fe2

Closes #440 Right now it will crash for whatever reason of using `?attr/` which is really annoying. I had this issue last time, can't remember how I fixed it. If you remove the custom attr from list_commits.xml, it will work. Atleast it's the case on my side. **Edit: fixed the crash.** --- Original Pull att :https://gitea.com/gitnex/GitNex/pulls/452 Merged with: 195bfc8fe244e8a3c67c3b7b9e31e9f5762ffa87
mmarif added this to the 3.0.0 milestone 2 years ago
mmarif added the
🚀 Improvement
🪒 Refactor
labels 2 years ago
mmarif closed this issue 2 years ago
mmarif was assigned by 6543 2 years ago
Collaborator
@mmarif -> 5807faed81138e48887813d95199c3f401aec469
Collaborator

@mmarif

Right now it will crash for whatever reason of using ?attr/ which is really annoying.

I know this may be too late, but i think youre passing the wrong context to the adapter when using getApplicationContext().

new CommitsAdapter(getApplicationContext())

Try passing CommitsActivity.this instead. It should work fine.

@mmarif > Right now it will crash for whatever reason of using ?attr/ which is really annoying. I know this may be too late, but i think youre passing the wrong context to the adapter when using getApplicationContext(). ``` new CommitsAdapter(getApplicationContext()) ``` Try passing ``CommitsActivity.this`` instead. It should work fine.
Poster
Owner

I know this may be too late, but i think youre passing the wrong context to the adapter when using getApplicationContext().

Yes, that's true I was passing the wrong context. Sometime just too many things to take care of. 😄

> I know this may be too late, but i think youre passing the wrong context to the adapter when using getApplicationContext(). Yes, that's true I was passing the wrong context. Sometime just too many things to take care of. :smile:
Poster
Owner

@mmarif -> 5807faed81

I saw you used < in somewhere, I guess in the mainactivity. That should also be changed?

> @mmarif -> 5807faed81 I saw you used `<` in somewhere, I guess in the mainactivity. That should also be changed?
Collaborator

@mmarif THIS is a other task - I'll look at all version checks in my refactor - just give me some time - I have a longer and longer ToDo list :(

this here is ready so I'll merge 🚀

@mmarif THIS is a other task - I'll look at all version checks in my refactor - just give me some time - I have a longer and longer ToDo list :( this here is ready so I'll merge :rocket:
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: gitnex/GitNex#452
Loading…
There is no content yet.