Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

3.7b3 update Notice: Undefined property: stdClass::$client_id in libraries\vendor\joomla\utilities\src\ArrayHelper.php on line 521 and 522 #14312

Merged
merged 2 commits into from Mar 2, 2017

Conversation

wojsmol
Copy link
Contributor

@wojsmol wojsmol commented Mar 2, 2017

Pull Request for Issues #14271 and #13624

Summary of Changes

Add COLUMN in file administrator/components/com_admin/sql/updates/mysql/3.7.0-2016-11-19.sql to JSchemaChangeitemMysql detect correctly the first query.

Testing Instructions

  1. Install latest staging
  2. Login to back-end and go to Extensions > Manage > Database > Other Information tab - note number of checked cheanges
  3. Apply patch
  4. Go to Extensions > Manage > Database > Other Information tab - note number of checked changes. Observe that has been detected one more change on the database structure.

Expected result

140 database changes were checked successfully. in database fix view

Actual result

139 database changes were checked successfully. in database fix view

Documentation Changes Required

N/A

@@ -1,3 +1,3 @@
ALTER TABLE `#__menu_types` ADD `client_id` int(11) NOT NULL DEFAULT 0;
ALTER TABLE `#__menu_types` ADD COLIMN `client_id` int(11) NOT NULL DEFAULT 0;
Copy link
Contributor

Choose a reason for hiding this comment

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

typo ? COLUMN

Copy link
Contributor Author

Choose a reason for hiding this comment

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

typo fixed in 23b30f0

@810
Copy link
Contributor

810 commented Mar 2, 2017

I have tested this item ✅ successfully on 23b30f0


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14312.

@wilsonge wilsonge merged commit d61934b into joomla:staging Mar 2, 2017
@wilsonge wilsonge added this to the Joomla 3.7.0 milestone Mar 2, 2017
@wojsmol wojsmol deleted the fix14271 branch March 2, 2017 22:04
jjaracz pushed a commit to jjaracz/joomla-cms that referenced this pull request Mar 9, 2017
…aries\vendor\joomla\utilities\src\ArrayHelper.php on line 521 and 522 (joomla#14312)

* Simple fix for joomla#14271

* fix typo
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.

None yet

4 participants