abika (Alex Bikadorov)
Animal

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Jun 20 2016, 6:53 PM (44 w, 23 h)
Availability
Available

Recent Activity

Sun, Apr 23

abika added a comment to D5493: New detailed tooltip for name column in panel view.

Thanks Martin!

Sun, Apr 23, 4:55 PM

Tue, Apr 18

abika updated the test plan for D5493: New detailed tooltip for name column in panel view.
Tue, Apr 18, 2:19 PM
abika updated the summary of D5493: New detailed tooltip for name column in panel view.
Tue, Apr 18, 2:17 PM
abika created D5493: New detailed tooltip for name column in panel view.
Tue, Apr 18, 2:10 PM

Wed, Apr 12

abika moved T5716: New release: 2.6.0 from TODO to Done on the Krusader board.
Wed, Apr 12, 4:37 PM · Krusader
abika added a comment to T5716: New release: 2.6.0.

Done. Somebody has to add 2.6.0 to the release list on bugzilla. I still don't have permission.

Wed, Apr 12, 4:36 PM · Krusader

Tue, Apr 11

abika edited the content of Krusader.
Tue, Apr 11, 6:05 PM · Krusader
abika closed D5292: Fix for crash when closing currently refreshing panel.

I took the liberty and commited your patch.

Tue, Apr 11, 5:41 PM · Krusader

Sun, Apr 9

abika updated subscribers of T5716: New release: 2.6.0.

Tomorrow or Tuesday - hopefully.
@palant If you want to be included in the author list, added yourself to the main.cpp.

Sun, Apr 9, 5:23 PM · Krusader
abika edited the content of Krusader.
Sun, Apr 9, 5:16 PM · Krusader
abika added a comment to D5292: Fix for crash when closing currently refreshing panel.

Actually the ListPanelFunc object is destroyed before the ListPanel, so when you do the check for this->panel, this points to already freed memory.

Sun, Apr 9, 4:37 PM · Krusader

Sat, Apr 8

abika renamed T5716: New release: 2.6.0 from "New release: 2.5.1" to "New release: 2.6.0".
Sat, Apr 8, 4:38 PM · Krusader
abika added a comment to T5716: New release: 2.6.0.

I agree. Probably lame but how about "Job-Man Show"? :)

Sat, Apr 8, 4:38 PM · Krusader
abika added a comment to D5292: Fix for crash when closing currently refreshing panel.

Ok now?

Sat, Apr 8, 3:42 PM · Krusader
abika updated the diff for D5292: Fix for crash when closing currently refreshing panel.
  • Fix for crash when closing currently refreshing panel
  • Safer version of "Fix for crash when closing currently refreshing panel" (f01640db)
Sat, Apr 8, 3:40 PM · Krusader

Wed, Apr 5

abika added a comment to D5292: Fix for crash when closing currently refreshing panel.

The code was ugly, but accessing a deleted object results in undefinefd behaviour.

Wed, Apr 5, 9:01 PM · Krusader

Tue, Apr 4

abika accepted D5292: Fix for crash when closing currently refreshing panel.

There has been code in place to prevent this, but looking at PanelManager::deletePanel(), the panel is now unconditionally deleted.
So a proper fix would be to re-introduce the previous behaviour.

Tue, Apr 4, 2:33 PM · Krusader

Fri, Mar 31

abika added a comment to D5165: Avoiding root mode.

Can someone please give an opinion on this?

Fri, Mar 31, 7:57 PM
abika accepted D5207: Port kde4_add_app_icon to ecm_add_app_icon.

Same here: can compile but not test. I guess, its fine.

Fri, Mar 31, 7:37 PM · Krusader

Mar 24 2017

abika created D5165: Avoiding root mode.
Mar 24 2017, 3:02 PM

Mar 20 2017

abika created T5716: New release: 2.6.0.
Mar 20 2017, 10:10 PM · Krusader

Mar 16 2017

abika closed T5673: Fix unsafe lambda usage as "Resolved".

Done with 5a26343a.

Mar 16 2017, 12:06 PM · Krusader
abika added a comment to T5673: Fix unsafe lambda usage.

Thanks for pointing this out! It's my code so I will fix it right away.

Mar 16 2017, 11:53 AM · Krusader

Mar 14 2017

abika added a comment to D4994: Rework of checksum dialogs..

Thanks! Typos fixed

Mar 14 2017, 8:33 PM

Mar 12 2017

abika edited the content of Krusader.
Mar 12 2017, 8:11 PM · Krusader

Mar 9 2017

abika added a comment to D4725: [SECURITY] Do not configure kdesu path and remove kdesudo support again.

Actually, there are some not so far away: KDE Partition Manager, K3b and KFloppy.

Mar 9 2017, 7:12 PM
abika created D4994: Rework of checksum dialogs..
Mar 9 2017, 6:41 PM

Mar 1 2017

abika added a comment to D4146: ListPanel: don't go back in history if refresh failed.

FYI: https://bugs.kde.org/show_bug.cgi?id=298292

Mar 1 2017, 7:00 PM · Krusader

Feb 27 2017

abika added a comment to D4725: [SECURITY] Do not configure kdesu path and remove kdesudo support again.

I wouldn't say this is "overreacting".

  • The security flaw is real. An attacker can probably get root access by exploiting Krusaders root mode
  • Krusader is the only GUI application that comes with an "execute as root" feature that I know of (anything else?). And yes, there is a reason for it.
  • Again: it is still possible to execute Krusader as root. But we are not advertising it anymore. Those who really need this can still create the desktop entry/action.
Feb 27 2017, 7:54 PM
abika added a comment to D4747: Rework of size/space calculation.

As a next step, if I didn't miss anything, calcSpaceFileSystem in AbstractJobThread::countLocalFiles() could be removed.
The packjob should be passed a list of URLs instead of file names which would eliminate the need to determine the URLs.

Feb 27 2017, 4:31 PM
abika added a comment to D4725: [SECURITY] Do not configure kdesu path and remove kdesudo support again.

Just found something: in krservices "kdesu" is still defined for tools and never used.

Feb 27 2017, 4:04 PM
abika accepted D4735: [SECURITY] Do not assume that kdesu is in PATH for user actions.

Thanks!

Feb 27 2017, 3:54 PM
abika accepted D4734: [SECURITY] Do not configure Krusader path, this is unnecessary.

Ship it!

Feb 27 2017, 3:44 PM
abika accepted D4725: [SECURITY] Do not configure kdesu path and remove kdesudo support again.
Feb 27 2017, 3:41 PM

Feb 23 2017

abika added inline comments to D4725: [SECURITY] Do not configure kdesu path and remove kdesudo support again.
Feb 23 2017, 8:16 PM
abika created D4750: Entroducing... lazy panels.
Feb 23 2017, 7:05 PM
abika created D4747: Rework of size/space calculation.
Feb 23 2017, 4:04 PM
abika added a comment to D4725: [SECURITY] Do not configure kdesu path and remove kdesudo support again.

Wait, screw that. I mixed up arguments.

Feb 23 2017, 3:50 PM
abika requested changes to D4725: [SECURITY] Do not configure kdesu path and remove kdesudo support again.

TL;DR: I agree with Wladimir and would recommend to remove Krusader root mode altogether because we shouldn't support unsafe usage.

Feb 23 2017, 3:30 PM

Feb 20 2017

abika requested review of D3611: Rework of job start behaviour. With new field for forcing enqueue, start or delay..
Feb 20 2017, 5:43 PM
abika updated the summary of D3611: Rework of job start behaviour. With new field for forcing enqueue, start or delay..
Feb 20 2017, 5:40 PM
abika added a comment to D3611: Rework of job start behaviour. With new field for forcing enqueue, start or delay..

Ok, coming back to this. I needed some time to re-think it.

Feb 20 2017, 5:38 PM
abika updated the diff for D3611: Rework of job start behaviour. With new field for forcing enqueue, start or delay..
  • Dialogs: merged "delay" checkbox with "reverser queue" button in copy/move dialogs
  • JobMan: improved coding style and documentation
  • JobMan: run next job not started yet even if queue mode is off
Feb 20 2017, 5:33 PM

Feb 17 2017

abika accepted D4645: Automatically configure kdesu or kdesudo, document these tools and kde-cli-tools as optional dependencies.

Thanks, works fine. Question is why Ubuntu is having a different tool doing the same thing.

Feb 17 2017, 5:48 PM
abika added a comment to D4610: Panel: Do not select .. in a filtered view if there are other entries.

Added a small fix: _count > 0 instead of _count > 1; the dummy file itself is not counted.

Feb 17 2017, 5:43 PM
abika closed T5372: Adding category for Krusader User actions as "Resolved".

Great! Thanks again for the quick response.

Feb 17 2017, 3:36 PM · KDE Store

Feb 16 2017

abika created T5372: Adding category for Krusader User actions.
Feb 16 2017, 4:25 PM · KDE Store

Feb 15 2017

abika abandoned D2857: Use KRun for "Open with"->"Other..." dialog.
In D2857#85422, @asensi wrote:

Is the working directory really important for (any) application?

I don't recall using any, although probably some exist, and if ListPanelFunc::runCommand() works and allows that those application work correctly, probably it would be better for Krusader to use that code.

Feb 15 2017, 6:08 PM
abika added a comment to D4610: Panel: Do not select .. in a filtered view if there are other entries.

I don't want to be pedantic but there is a small misbehavior: when the current item is ".." and "Show hidden files" setting is changed (Alt+".") the current jumps to the first file now. Probably nobody will ever notice but checking against empty _quickFilterMask should also solve it.

Feb 15 2017, 5:30 PM
abika added a comment to D4512: Rework of vfile and KrPermHandler class.

Thank you testing.

I couldn't find any regressions.
seems to work correctly

Feb 15 2017, 5:01 PM

Feb 14 2017

abika retitled D4609: Improved integration of (quick) search bar in panel from to Improved integration of (quick) search bar in panel.
Feb 14 2017, 4:07 PM

Feb 13 2017

abika closed D4433: #375994 - List kio-extras under dependencies as recommended package.

Merged with master

Feb 13 2017, 8:56 PM · Krusader
abika accepted D4433: #375994 - List kio-extras under dependencies as recommended package.
Feb 13 2017, 8:55 PM · Krusader
abika closed D4427: Update "get involved" page to indicate how one can register.

Merged with master

Feb 13 2017, 8:54 PM · Websites, Krusader
abika closed D4420: #375894 - Use case-insensitive as default for text search.

Merged with master

Feb 13 2017, 8:54 PM · Krusader
abika closed D4419: #375830 - Don't send SIGINT to terminal emulator unnecessarily.

Merged with master

Feb 13 2017, 8:54 PM · Krusader
abika closed D4418: #314258 - Duplicate tab settings when duplicating tabs.

Merged with master

Feb 13 2017, 8:54 PM · Krusader
abika closed D4417: #375831 - Sort empty file extensions correctly.

Merged with master (and Phabricator fails again to autoclose this)

Feb 13 2017, 8:53 PM · Krusader
abika edited the content of Krusader.
Feb 13 2017, 7:02 PM · Krusader

Feb 10 2017

abika added a comment to D2857: Use KRun for "Open with"->"Other..." dialog.

I see. Is the working directory really important for (any) application? It seems strange that Krusader does this different compared to e.g. Dolphin.

Feb 10 2017, 6:03 PM
abika accepted D4418: #314258 - Duplicate tab settings when duplicating tabs.

Could it be that the issue there was being caused by the extra ListPanel::start() call?

Quick test says: yes!

Feb 10 2017, 4:16 PM · Krusader

Feb 9 2017

abika added a comment to D4418: #314258 - Duplicate tab settings when duplicating tabs.

Ok, I see now that this is also not a good solution. The settings are going down to the PanelPopup and KrInterviews. All this should be copied. Sorry I didn't got this earlier.

Feb 9 2017, 8:44 PM · Krusader

Feb 8 2017

abika retitled D4512: Rework of vfile and KrPermHandler class from to Rework of vfile and KrPermHandler class.
Feb 8 2017, 8:27 PM
abika accepted D4146: ListPanel: don't go back in history if refresh failed.

I personally prefer the old behavior. After restart going back to the "nearest" parent folder of a deleted path was pretty nice. And if it was a removable device I can mount it and go back to the folder with the history.

Feb 8 2017, 3:20 PM · Krusader
abika accepted D4420: #375894 - Use case-insensitive as default for text search.

Like I said, I don't really care. In KFind case sensitivity is also off by default.

Feb 8 2017, 3:02 PM · Krusader

Feb 4 2017

abika added a comment to D4418: #314258 - Duplicate tab settings when duplicating tabs.

Hmm, works fine but using a temporary config group seems a bit hackish.

Feb 4 2017, 2:13 PM · Krusader
abika added a comment to D4420: #375894 - Use case-insensitive as default for text search.

I personally don't care. Default search with grep is case sensitive. Any other opinions?

Feb 4 2017, 2:10 PM · Krusader
abika accepted D4419: #375830 - Don't send SIGINT to terminal emulator unnecessarily.

Thanks for the fix!

Feb 4 2017, 2:09 PM · Krusader
abika accepted D4427: Update "get involved" page to indicate how one can register.
Feb 4 2017, 2:08 PM · Websites, Krusader
abika accepted D4417: #375831 - Sort empty file extensions correctly.
Feb 4 2017, 2:08 PM · Krusader

Jan 29 2017

abika added a comment to D4316: Panel/VFS: Rework of deletion/moveToTrash/virtualRemove of files and directories.
In D4316#81044, @asensi wrote:

About that F11 subject: Is it a change in the user interface of Krusader? :-?

Jan 29 2017, 2:35 PM

Jan 28 2017

abika added a comment to D4316: Panel/VFS: Rework of deletion/moveToTrash/virtualRemove of files and directories.

Thanks! Merged with master.

Jan 28 2017, 5:43 PM
abika accepted D4195: Changes in the installation documentation (2) - Web pages.

Sorry, missed it again. Changes are fine!

Jan 28 2017, 4:00 PM · Krusader, Websites

Jan 27 2017

abika retitled D4316: Panel/VFS: Rework of deletion/moveToTrash/virtualRemove of files and directories from to Panel/VFS: Rework of deletion/moveToTrash/virtualRemove of files and directories.
Jan 27 2017, 6:19 PM

Jan 25 2017

abika edited the content of Krusader.
Jan 25 2017, 6:48 PM · Krusader

Jan 20 2017

abika edited the content of Krusader.
Jan 20 2017, 8:00 PM · Krusader

Jan 18 2017

abika edited the content of Krusader.
Jan 18 2017, 4:24 PM · Krusader

Jan 17 2017

abika added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .

I would not recommend the information of that page, a line like

su -c "make install"

won't work for Debian/Mint/Ubuntu/Kubuntu/etc. users.

Jan 17 2017, 8:58 PM · Krusader, Websites
abika added a comment to D4008: A warning note, like the formerly used ones, was added to the installation section. .

The idea was to follow the Krusader handbook style (in docs.kde.org) to put out a big warning, in this case: "do not follow the next outdated information, it will cause you problems", "follow the linked file" :-)

Jan 17 2017, 3:52 PM · Krusader, Websites

Jan 16 2017

abika updated the diff for D3611: Rework of job start behaviour. With new field for forcing enqueue, start or delay..
  • Dialogs: merged "delay" checkbox with "reverser queue" button in copy/move dialogs
Jan 16 2017, 6:00 PM
abika requested changes to D4008: A warning note, like the formerly used ones, was added to the installation section. .

Hmm, sorry I don't think the different style fits to the rest of the page. There are already a lot of Note:s . We should change all of them. But why a single (big) one with different style?

Jan 16 2017, 2:46 PM · Krusader, Websites

Jan 15 2017

abika added a comment to D3994: add asynchronous vfs refresh.

I'm actually unsure if there is any benefit for a "real" async refresh. The panel currently depends on it and the Qt event loop prevents an UI freeze during a long refresh.

Jan 15 2017, 9:45 PM · Krusader
abika added a comment to T5057: Check usage of QDir::setCurrent().

I have no idea why the directory change is there in the code or why it could be important. Maybe remove it and see what happens(?)

Jan 15 2017, 9:41 PM · Krusader

Jan 11 2017

abika added a comment to T5062: Improve search results tab.

Great! Nice feature. Is that KWrite from KParts?

Jan 11 2017, 5:30 PM · Krusader
abika closed D4071: Rework of behaviour of actions for embedded terminal (splitter mode and fullscreen mode)..

Merged with master.

Jan 11 2017, 5:20 PM

Jan 10 2017

abika updated subscribers of D4071: Rework of behaviour of actions for embedded terminal (splitter mode and fullscreen mode)..
Jan 10 2017, 2:58 PM
abika retitled D4071: Rework of behaviour of actions for embedded terminal (splitter mode and fullscreen mode). from to Rework of behaviour of actions for embedded terminal (splitter mode and fullscreen mode)..
Jan 10 2017, 2:54 PM

Jan 7 2017

abika added a comment to T5038: Update documentation links in order to point to the new Krusader 2.5.0 handbook.

Sorry Tonny. It just happened that we worked on it at the same time.

Jan 7 2017, 8:39 PM · Krusader
abika added a comment to T5038: Update documentation links in order to point to the new Krusader 2.5.0 handbook.

... or better: I already did. Check the website.

Jan 7 2017, 6:20 PM · Krusader
abika added a comment to T5038: Update documentation links in order to point to the new Krusader 2.5.0 handbook.

I'm currently doing this :/

Jan 7 2017, 6:20 PM · Krusader

Jan 4 2017

abika edited the content of Krusader.
Jan 4 2017, 4:32 PM · Krusader
abika closed D3938: VFS: cleaner code for refresh check on vfs operation without current directory.

And sorry for the terrible long commit messages. arc is still driving me nuts. Automatic merge and commit messages I totally don't want but somehow no automatic differential closing with --merge.

Jan 4 2017, 3:39 PM

Jan 3 2017

abika added a comment to T4993: Fix links in website footer.

If you think everything is resolved please close this.

Jan 3 2017, 8:48 PM · Krusader
abika added a comment to T3419: Rework of VFS.

Oh, the job creation in vfs::calcSpaceKIO() is working now! I could swear that there was an Qt error message the last time I tried, saying something about not being able to create an QObject with a parent in a different thread.

Jan 3 2017, 8:41 PM · Krusader
abika updated D3938: VFS: cleaner code for refresh check on vfs operation without current directory.
Jan 3 2017, 7:45 PM
abika retitled D3938: VFS: cleaner code for refresh check on vfs operation without current directory from to VFS: cleaner code for refresh check on vfs operation without current directory.
Jan 3 2017, 7:37 PM

Dec 29 2016

abika added a comment to T4993: Fix links in website footer.
  • projects.kde.org (actually links to kde.org/.../krusader, text should probably be different since projects went away)

Replaced with Phabricator link

Done

  • kde-files.org (should go to store.kde.org somewhere)

Removed. Krusader is not (yet?) included in the KDE store.

  • donate (this one's ok)

Removed. People should better donate to KDE e.v. instead.

Dec 29 2016, 7:08 PM · Krusader
abika accepted D3855: Add .reviewboardrc and .arcconfig.

Ship it!

Dec 29 2016, 6:37 PM
abika accepted D3856: Add includes..

Cannot test on BSD, but I guess its fine.

Dec 29 2016, 6:05 PM

Dec 14 2016

abika added a comment to T3419: Rework of VFS.

I noticed a minor issue with the toolbar. Since I don't use toolbars very often, I tried to move the Job toolbar to the bottom of the window, but after I restarted Krusader it was always at the top again. I noticed the same behavior with the Main toolbar, so it might be due to KXMLGuiWindow.

Dec 14 2016, 7:07 PM · Krusader