Skip to content

Block editor#4

Open
emfluenceindia wants to merge 20 commits into
masterfrom
block-editor
Open

Block editor#4
emfluenceindia wants to merge 20 commits into
masterfrom
block-editor

Conversation

@emfluenceindia

Copy link
Copy Markdown
Contributor

Phase 1: Plugin fixes for block widget editor.
Version: 2.13.1

[ x ] Add: Two new fields added. Form name and Form ID. Section: Form Identification.
[ x ] Fix: Group search was not working.
[ x ] Fix: Group Add button was not working.
[ x ] Fix: Group removal (deselecting a previously added group) was not working.

@jacobischwartz jacobischwartz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly code comment and escaping changes. Thanks @emfluenceindia!

Comment thread admin.php Outdated
<?php

// Subrata
// add_filter( 'use_widgets_block_editor', '__return_false' );

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's remove the commented code 😉

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment thread admin.php Outdated
'emfluence-emailer-widget'
,plugins_url( '/js/widget-settings.min.js', __FILE__ )
//Subrata
//,plugins_url( '/js/widget-settings.js', __FILE__ )

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commented code to remove

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment thread emfluence.php Outdated
Description: Easily add forms to your website for contacts to add or update their details in your emfluence Marketing Platform account.
Author: emfluence Digital Marketing
Version: 2.13
Version: 2.13.1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's increase this to 2.14

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment thread js/widget-settings.js Outdated

//Subrata
$groupIdsContainer = this.getGroupIdsContainer($widget);
//Subrata

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code comment to remove

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment thread js/widget-settings.js Outdated

//Subrata
//console.log(values.id);
//Subrata

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code comments to remove

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment thread widget.php Outdated
$output .= $this->form_template_custom_variables($defaults, $instance);
$output .= $this->form_template_notification($instance);

//Subrata: new fld

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove code comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment thread widget.php Outdated

// Output the datalist for groups just once
if( intval($this->number) == 0 ) {
// if( intval($this->number) == 0 )

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove commented code

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment thread widget.php Outdated
if( intval($this->number) == 0 ) {
// if( intval($this->number) == 0 )
if( intval($this->number) >= 0 ) { // Subrata
$output .= '<input type="hidden" id="emfluence_email_signup_instance" value="' . $this->number . '" />';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Escape variable in output

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment thread widget.php Outdated
}

// Clean up the free-form areas
// Subrata: new fld

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove code comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment thread widget.php Outdated

// If the current user isn't allowed to use unfiltered HTML, filter it
if ( !current_user_can('unfiltered_html') ) {
// Subrata: new fld

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove code comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@emfluenceindia

Copy link
Copy Markdown
Contributor Author

All changes are made.

@emfluenceindia

Copy link
Copy Markdown
Contributor Author

Hi Jacob,
All necessary changes from October 2023 are made. Kindly let me know if any more changes are required.

Regards,
Subrata

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants