profile
viewpoint
Slava Abakumov slaFFik @awesomemotive https://ovirium.com // Be good, have fun, create things.

CMB2/CMB2 2688

CMB2 is a developer's toolkit for building metaboxes, custom fields, and forms for WordPress that will blow your mind.

CMB2/CMB2-Snippet-Library 340

CMB2 Code Snippet Library

CMB2/cmb2-attached-posts 126

Custom field for CMB2 for attaching posts to a page.

CMB2/CMB2-Post-Search-field 61

Custom field for CMB2 which adds a post-search dialog for searching/attaching other post IDs

slaFFik/BP-Default-Data 13

Create lots of users, messages, friends connections, groups, topics, activity items in BuddyPress - useful for testing purpose.

CMB2/cmb2-group-map 12

Originally created for Post Status (https://poststatus.com/) to associate footnotes with notes, This CMB2 addon allows you to use CMB2 group fields to manage custom post type entries.

slaFFik/buddypress-groups-extras 12

Adding missing functionality to BuddyPress Groups.

slaFFik/github-auto-deploy 10

Deploy your Github projects instantly on any `push` to a specified branch.

CMB2/CMB2-Related-Links 9

Custom field type for CMB2 which adds a releated links repeatable group field.

slaFFik/BP-GTM-System 9

DEPRECATED. BuddyPress Group Tasks Management System will turn your site into a developer center, where tasks, projects, discussions, categories and tags will help you maintain products.

issue openedawesomemotive/wpforms-phpcs

Suggestion: remove requirement for a new line before break; statement inside switch

Expected Behavior

<!--- Tell us how it should work. --> The switch statement is already nicely formatted by default, break; is on the last line inside the case section most of the time. I would like to be able to save some space and not bloat the easily readable code with an extra new line requirement we currently have in effect. So I propose to allow this:

switch( $foo ) {
	case 'foo':
		$bar = 1;
		break;

	case 'bar':
		$bar = 2;
		break;

	case 'baz':
		$bar = 3;
		break;

	case 'krya':
	default:
		$bar = 4;
}

Current Behavior

<!--- Explain the difference from the current behavior. --> Right now wpforms-phpcs insists on having a new line before break, which looks like this:

switch( $foo ) {
	case 'foo':
		$bar = 1;

		break;

	case 'bar':
		$bar = 2;

		break;

	case 'baz':
		$bar = 3;

		break;

	case 'krya':
	default:
		$bar = 4;
}

created time in 4 days

issue openedspatie/ray

Popups should be on top of the Ray app

Describe the bug Popup which tells whether a new version available is hidden behind the Ray app itself. Same with the popup about the Ray version.

Those 2 should have a higher "z-index" (if speaking in CSS terms) to be on top of the Ray app because they are triggered manually and user expects them to see. But when the Ray app is placed exactly in the middle of the screen - it covers those small popups.

Versions Ray version (you can see this in "About Ray"): 1.19.0

To Reproduce Steps to reproduce the behavior:

  1. Go to Menu > About Ray
  2. Go to Menu > Check for updates
  3. observe that Ray app covers those 2 popups (maybe others as well).

Expected behavior Those small informational popups should have higher priority and be rendered on top of Ray even if Ray configured to be rendered on top of all other windows.

Screenshots

image

Desktop (please complete the following information):

  • OS: Windows 10 Pro 21H2

created time in 9 days

issue commentspatie/ray

Windows: app doesn't have publisher, not signed

It's not possible to reopen it :) There is no such button for regular users.

slaFFik

comment created time in 12 days

delete branch slaFFik/WordPress-Coding-Standards

delete branch : patch-1

delete time in 22 days

PR opened WordPress/WordPress-Coding-Standards

README.md: Update the minimum supported PHP version

In the readme.md file it is stated that PHP 5.2 is the minimum supported PHP version, while it's already PHP 5.6 for quite some time and technically WP does not work on PHP 5.2-5.5.

+2 -2

0 comment

1 changed file

pr created time in 23 days

push eventslaFFik/WordPress-Coding-Standards

Slava Abakumov

commit sha 0052562dee5e0c2229913e371d05cd925bba523d

Update the min support PHP version

view details

push time in 23 days

fork slaFFik/WordPress-Coding-Standards

PHP_CodeSniffer rules (sniffs) to enforce WordPress coding conventions

fork in 23 days

push eventslaFFik/lando-wordpress

Slava Abakumov

commit sha 3608466878c161ee66c83118cd4cb4cf45c29cf8

Upgrade PHP to 7.4

view details

push time in 25 days

startedk-samuel/faceted-search

started time in 25 days

fork slaFFik/butterbean

A neat little post meta framework.

fork in a month

startedLLoadout/mysqlcompare

started time in 2 months

push eventslaFFik/lando-wordpress

Pavlo Opanasenko

commit sha b0823d012c03cb581d3bac2ec14c8c8fa1a69201

Update README.md

view details

push time in 2 months

issue openedspatie/ray

Windows: incorrect notification title in Notifications Center

Describe the bug Internal app identifier is used as a notification title: electron.app.Ray

Versions Ray version (you can see this in "About Ray"): 1.18.4

Expected behavior The Windows notification should have a proper title, at least something like Ray.

Screenshots image

Desktop (please complete the following information):

  • OS: [e.g. macOS] Windows 10 Pro 21H2

created time in 2 months

issue commentwoocommerce/woocommerce-admin

PHP Warnings in several RemoteInboxNotifications/Transformers/* classes

And another event with the same warnings:

[19-Nov-2021 09:02:21 UTC] PHP Warning:  array_column() expects parameter 1 to be array, string given in /var/www/releases/433e3bedbfe02c6a80285dbdaff97b161f57d436/app/plugins/woocommerce/packages/woocommerce-admin/src/RemoteInboxNotifications/Transformers/ArrayColumn.php on line 27
[19-Nov-2021 09:02:21 UTC] PHP Warning:  count(): Parameter must be an array or an object that implements Countable in /var/www/releases/433e3bedbfe02c6a80285dbdaff97b161f57d436/app/plugins/woocommerce/packages/woocommerce-admin/src/RemoteInboxNotifications/Transformers/Count.php on line 24
slaFFik

comment created time in 2 months

startedCherryCircle/MySQLSlowLogAnalyzer

started time in 3 months

issue openedwoocommerce/woocommerce-admin

PHP Warnings in several RemoteInboxNotifications/Transformers/* classes

Describe the bug

<!-- A clear and concise description of what the bug is. Please be as descriptive as possible. -->

PHP Warnings are generated:

[28-Oct-2021 00:25:11 UTC] PHP Warning:  array_column() expects parameter 1 to be array, string given in /var/www/releases/f6ed0bce20e9e77a9c24b0264768de8562188d86/app/plugins/woocommerce/packages/woocommerce-admin/src/RemoteInboxNotifications/Transformers/ArrayColumn.php on line 27
[28-Oct-2021 00:25:11 UTC] PHP Warning:  count(): Parameter must be an array or an object that implements Countable in /var/www/releases/f6ed0bce20e9e77a9c24b0264768de8562188d86/app/plugins/woocommerce/packages/woocommerce-admin/src/RemoteInboxNotifications/Transformers/Count.php on line 24

To Reproduce

<!-- Describe the steps to reproduce the behavior.-->

Unfortunately, I can't provide exact steps to replicate the problem.

Expected behavior

<!-- A clear and concise description of what you expected to happen. --> There should be no PHP warnings in any case.

Additional context

<!--Any additional context or details you think might be helpful.--> <!--Ticket numbers/links, plugin versions, system statuses etc.--> WordPress 5.8.1, WooCommerce 5.8.0.

created time in 3 months

more